-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closed
treeverse/dvc-azure
#37Labels
bugDid we break something?Did we break something?fs: azureRelated to the Azure filesystemRelated to the Azure filesystemp1-importantImportant, aka current backlog of things to doImportant, aka current backlog of things to doregressionOhh, we broke something :-(Ohh, we broke something :-(
Description
There are 2 bugs:
-
Reimplement fs.info() to not depend on fs.ls() fsspec/adlfs#390
Requires adlfs bump. -
rm: Ensure
expand_path=False. dvc-azure#37
Other filesystems do nothing ifrecursive=Falsebut adlfs has a custom_expand_pathimplementation that fails when the directory doesn't exist even if we passrecursive=False, so fs: local: Accept list of paths inrm. dvc-objects#176 didn't fix.
Could probably be also fixed upstream (adlfs) but we still want to explicitly passexpand_path=Falsefor performance reasons.
Metadata
Metadata
Assignees
Labels
bugDid we break something?Did we break something?fs: azureRelated to the Azure filesystemRelated to the Azure filesystemp1-importantImportant, aka current backlog of things to doImportant, aka current backlog of things to doregressionOhh, we broke something :-(Ohh, we broke something :-(