Skip to content

Remove old shard key mapping format#7047

Merged
timvisee merged 3 commits intodevfrom
remove-old-shard-key-map-format
Oct 29, 2025
Merged

Remove old shard key mapping format#7047
timvisee merged 3 commits intodevfrom
remove-old-shard-key-map-format

Conversation

@timvisee
Copy link
Member

@timvisee timvisee commented Aug 13, 2025

In #5838 and #6209 we switched to a new shard key mapping format on disk. This was necessary for compatibility with numeric keys.

Support for the old format was kept to allow forward/backward compatibility. Qdrant 1.14 forces the new format and migrates the file if needed. The plan was to remove support for the old format in Qdrant 1.15.

This now removes the old format. The change will land in 1.16.0.

Tasks

All Submissions:

  • Contributions should target the dev branch. Did you create your branch from dev?
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  1. Does your submission pass tests?
  2. Have you formatted your code locally using cargo +nightly fmt --all command prior to submission?
  3. Have you checked your code using cargo clippy --all --all-features command?

@timvisee

This comment was marked as resolved.

@timvisee timvisee added the do not merge Pull requests blocked on an external event label Aug 13, 2025
@timvisee timvisee force-pushed the remove-old-shard-key-map-format branch 3 times, most recently from 3152fa6 to 20ab093 Compare August 22, 2025 09:27
@timvisee timvisee added release:1.16.0 Pull requests that should be merged for the Qdrant 1.16.0 release. and removed do not merge Pull requests blocked on an external event labels Oct 21, 2025
@timvisee timvisee force-pushed the remove-old-shard-key-map-format branch from 20ab093 to d4a8885 Compare October 21, 2025 09:18
@timvisee timvisee marked this pull request as ready for review October 21, 2025 12:08
coderabbitai[bot]

This comment was marked as resolved.

@qdrant qdrant deleted a comment from coderabbitai bot Oct 21, 2025
@timvisee timvisee merged commit c64a297 into dev Oct 29, 2025
15 checks passed
@timvisee timvisee deleted the remove-old-shard-key-map-format branch October 29, 2025 16:36
timvisee added a commit that referenced this pull request Nov 14, 2025
* Remove old shard key mapping format variant

* Change single variant enum into tuple struct

* Remove migration logic for old to new shard key format
timvisee added a commit that referenced this pull request Nov 19, 2025
This reverts commit 783c128.

This reverts pull request <#7047>.
timvisee added a commit that referenced this pull request Nov 19, 2025
This reverts commit 783c128.

This reverts pull request <#7047>.
timvisee added a commit that referenced this pull request Nov 19, 2025
…7564)

* Revert "Remove old shard key mapping format (#7047)"

This reverts commit 783c128.

This reverts pull request <#7047>.

* Bump TODOs
timvisee added a commit that referenced this pull request Nov 25, 2025
…7564)

* Revert "Remove old shard key mapping format (#7047)"

This reverts commit 783c128.

This reverts pull request <#7047>.

* Bump TODOs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:1.16.0 Pull requests that should be merged for the Qdrant 1.16.0 release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants