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

index: add db-based implementation (e.g. sqlite) #208

Closed
efiop opened this issue Oct 18, 2022 · 1 comment · Fixed by #248
Closed

index: add db-based implementation (e.g. sqlite) #208

efiop opened this issue Oct 18, 2022 · 1 comment · Fixed by #248
Assignees

Comments

@efiop
Copy link
Member

efiop commented Oct 18, 2022

We currently use in-memory prefix trie, but for large enough indexes it will be much nicer to have an ability to use a proper db and also use it to do operations like diff more efficiently (e.g. directly in sql query instead of fetching and comparing in python).

@efiop efiop self-assigned this Nov 29, 2022
efiop added a commit to efiop/dvc-data that referenced this issue Jan 5, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 6, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 7, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 8, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 8, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 10, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 11, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 11, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 12, 2023
efiop added a commit to efiop/dvc-data that referenced this issue Jan 15, 2023
Temporary connector before persistent index iterative#208 is fully supported.

Pre-requisite for iterative/dvc#8761
efiop added a commit that referenced this issue Jan 15, 2023
Temporary connector before persistent index #208 is fully supported.

Pre-requisite for iterative/dvc#8761
efiop added a commit that referenced this issue Jan 15, 2023
efiop added a commit to iterative/dvc that referenced this issue Jan 16, 2023
@efiop
Copy link
Member Author

efiop commented Jan 16, 2023

For the record, DataIndex now has an open method that allows using sqlite-based prefix tree. The current implementation is not perfect by any means(e.g. sqltrie is pretty raw and will need fixes/optimizations along the way), but sets course in a proper direction.

efiop added a commit to iterative/dvc that referenced this issue Jan 30, 2023
efiop added a commit to iterative/dvc that referenced this issue Jan 31, 2023
efiop added a commit to iterative/dvc that referenced this issue Jan 31, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 3, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 3, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 3, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 4, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 4, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 5, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 6, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 6, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 6, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 6, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 6, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 7, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 7, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 7, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 7, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 7, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 7, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 8, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 9, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 9, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 9, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 9, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 12, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 12, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 13, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 13, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 13, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 13, 2023
efiop added a commit to iterative/dvc that referenced this issue Feb 13, 2023
daavoo pushed a commit to iterative/dvc that referenced this issue Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant