Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dvcfs: base on fsspec #7521

Merged
merged 3 commits into from
Mar 31, 2022
Merged

dvcfs: base on fsspec #7521

merged 3 commits into from
Mar 31, 2022

Conversation

efiop
Copy link
Contributor

@efiop efiop commented Mar 30, 2022

Initial migration of dvcfs to fsspec. Pre-requisite for making repofs fsspec-compatible and following dvc-object/data extraction.

@efiop efiop requested a review from a team as a code owner March 30, 2022 09:09
@efiop efiop requested a review from skshetry March 30, 2022 09:09
@efiop efiop force-pushed the dvcfs-ls branch 2 times, most recently from 102df85 to 9e8515b Compare March 30, 2022 19:52
@efiop efiop changed the title [WIP] repofs: use dvcfs.ls instead of dvcfs.walk dvcfs: base on fsspec Mar 30, 2022
@efiop efiop changed the title dvcfs: base on fsspec [WIP] dvcfs: base on fsspec Mar 30, 2022
@efiop efiop force-pushed the dvcfs-ls branch 3 times, most recently from 8466060 to 23c630b Compare March 31, 2022 08:49
efiop added 3 commits March 31, 2022 17:30
On our way to making dvcfs fsspec-compliant.
Using ls instead of walk reduces potential problems that may occur if
dvcfs.walk happens to yield results in different order than
gitfs/localfs.walk.
@efiop efiop changed the title [WIP] dvcfs: base on fsspec dvcfs: base on fsspec Mar 31, 2022
@efiop efiop added the refactoring Factoring and re-factoring label Mar 31, 2022
@efiop efiop merged commit d600eec into iterative:main Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Factoring and re-factoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant