fix: remove index.knn.space_type setting#141
Conversation
WalkthroughRemoved the explicit index.knn.space_type="cosinesimil" from CreateCollectionAsync’s index creation in OpenSearchVectorDatabase.cs, leaving index.knn enabled and vector mapping unchanged. The index now relies on the default KNN space type. No public API signatures or control flow were altered. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Hey @HavenDV Looks like the build ci/cd failed to push the new nuget package. Could you please fix that when you have a chance? |
This "index.knn.space_type" setting is no longer available in opensearch 3. Rather than creating the index, it throws an exception. Cosine Similarity was already being specified in the mapping, so the setting was not required in older versions anyway. I removed the offending line to restore functionality for version 3 of opensearch.
Summary by CodeRabbit
Refactor
Chores
Notes