FE-611 | Add Vector Index Feature #21793
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cmyk47
previously requested changes
Jun 2, 2025
...ps/system/_admin/aardvark/APP/react/src/views/collections/indices/useSupportedIndexTypes.tsx
Outdated
Show resolved
Hide resolved
.../system/_admin/aardvark/APP/react/src/views/collections/indices/CollectionIndicesContext.tsx
Outdated
Show resolved
Hide resolved
...pps/system/_admin/aardvark/APP/react/src/views/collections/indices/addIndex/AddIndexForm.tsx
Outdated
Show resolved
Hide resolved
...pps/system/_admin/aardvark/APP/react/src/views/collections/indices/addIndex/AddIndexForm.tsx
Outdated
Show resolved
Hide resolved
...ps/system/_admin/aardvark/APP/react/src/views/collections/indices/useSupportedIndexTypes.tsx
Outdated
Show resolved
Hide resolved
…ture/FE-611/add-new-vector-index-ui
KVS85
previously requested changes
Jun 5, 2025
...ps/system/_admin/aardvark/APP/react/src/views/collections/indices/useSupportedIndexTypes.tsx
Show resolved
Hide resolved
…ture/FE-611/add-new-vector-index-ui
Simran-B
reviewed
Jun 6, 2025
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Show resolved
Hide resolved
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
…ture/FE-611/add-new-vector-index-ui
Simran-B
reviewed
Jun 10, 2025
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
Simran-B
reviewed
Jun 10, 2025
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
Simran-B
reviewed
Jun 10, 2025
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
Simran-B
reviewed
Jun 10, 2025
...ardvark/APP/react/src/views/collections/indices/addIndex/vectorIndex/useCreateVectorIndex.ts
Outdated
Show resolved
Hide resolved
Simran-B
previously requested changes
Jun 10, 2025
There was a problem hiding this comment.
The vector index supports the following options (not persisted / returned by the server), similar to other index types:
inBackground(see below)parallelism: The number of threads to use for indexing. The default is2.
The only other index with a parallelism option seems to be the inverted index but we don't display it in the UI - I'm not sure if this is a conscience decision or not. I believe we found that 2 is a bit faster than 1 but any larger number were performing worse than 2, so there is little point in changing it for the inverted index. It could be very different for the vector index, though.
I noticed that e.g. for the persistent index, we don't have a particularly useful tooltip for Create in background:
My suggestion is to change it everywhere to something like this:
Enable this option to keep the collection/shards available for write operations by not using an exclusive write lock for the duration of the index creation.
…ture/FE-611/add-new-vector-index-ui
Confirmed with the jira reporter and it was suggested that the label should be "beta".
…ture/FE-611/add-new-vector-index-ui
…ew-vector-index-ui # Conflicts: # CHANGELOG
mchacki
pushed a commit
that referenced
this pull request
Jun 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Scope & Purpose
Adds support for the new experimental vector index to the collection Indexes view in the core DB UI. The index appears as
"Vector index (beta)"in the dropdown, and is only shown when the server is started with the--experimental-vector-indexflag.Checklist
Related Information
Note: This backend PR addresses a bug where the vector index appears even without the
--experimental-vector-indexflag. This PR can proceed with a mention of the backend PR as an upstream dependency.