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

Add tooling back on 9.10.x branch to generate int7_hnsw.9.10.zip bwc index #13879

Open
wants to merge 1 commit into
base: branch_9_10
Choose a base branch
from

Conversation

mikemccand
Copy link
Member

Spinoff from #13867 to add the tooling back onto branch_9_10 in case we ever need to regenerate the int7_hnsw.9.10.0.zip bwc index again.

@@ -82,6 +82,16 @@ public void testCreateSortedIndex() throws IOException {
sortedTest.createBWCIndex();
}

public void testCreateInt7HNSWIndices() throws IOException {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we add this to the back compat script? And if so then there could be a collision with future versions.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm I think we can just rename int8_hnsw -> int7_hnsw in addBackCompatIndexes.py? But I would do that change on our latest branches (9.12.x, 10.x, 10.0.x) as a separate follow-on PR? This PR is just targetting 9.10.x branch in case we ever need to regen 9.10.x bwc indices again (hope not!).

Copy link

This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the [email protected] list. Thank you for your contribution!

@github-actions github-actions bot added the Stale label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants