Skip to content

CXXCBC-516: Return feature_not_available when upserting vector index to incompatible cluster#572

Merged
DemetrisChr merged 1 commit intocouchbase:mainfrom
DemetrisChr:CXXCBC-516-vector-idx-fna
May 24, 2024
Merged

CXXCBC-516: Return feature_not_available when upserting vector index to incompatible cluster#572
DemetrisChr merged 1 commit intocouchbase:mainfrom
DemetrisChr:CXXCBC-516-vector-idx-fna

Conversation

@DemetrisChr
Copy link
Copy Markdown
Contributor

Changes

  • Add is_vector_index() method to core::management::search::index that checks whether any fields in the index definition have type "vector" or "vector_base64".
  • When upserting a search index, check whether the index is a vector index and check whether the vector search cluster capability is present. If not, return the feature_not_available error code.
  • Add unit tests for is_vector_index() and an integration test for the capability check.
  • Move the search index management integration tests to a separate file.

Results

All tests pass

@DemetrisChr DemetrisChr force-pushed the CXXCBC-516-vector-idx-fna branch 2 times, most recently from f1b01c6 to c871d48 Compare May 23, 2024 14:20
@DemetrisChr DemetrisChr force-pushed the CXXCBC-516-vector-idx-fna branch from c871d48 to 3d66b88 Compare May 23, 2024 15:58
@avsej avsej force-pushed the CXXCBC-516-vector-idx-fna branch from 3d66b88 to 4cf9254 Compare May 24, 2024 01:59
@DemetrisChr DemetrisChr merged commit f72a089 into couchbase:main May 24, 2024
@DemetrisChr DemetrisChr deleted the CXXCBC-516-vector-idx-fna branch May 24, 2024 08:54
avsej added a commit that referenced this pull request May 24, 2024
* origin/main:
  CXXCBC-516: Return feature_not_available when upserting vector search index to incompatible cluster (#572)
  CXXCBC-509: Add feature flag for zone-aware replica reads (#571)
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 this pull request may close these issues.

2 participants