Skip to content

Add reconfigure options for Dynamic index#406

Merged
bevzzz merged 4 commits intomainfrom
fix/reconfigure-dynamic
Feb 27, 2026
Merged

Add reconfigure options for Dynamic index#406
bevzzz merged 4 commits intomainfrom
fix/reconfigure-dynamic

Conversation

@bevzzz
Copy link
Collaborator

@bevzzz bevzzz commented Feb 26, 2026

Add reconfigure.dynamic and fix some typos.

Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@bevzzz bevzzz changed the title Fix/reconfigure dynamic Add reconfigure options for Dynamic index Feb 26, 2026
@dirkkul dirkkul requested a review from Copilot February 27, 2026 07:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds reconfiguration support for dynamic vector indexes and fixes a typo in the type name VectorIndexConfigDymamicUpdate (corrected to VectorIndexConfigDynamicUpdate). The PR extends the reconfigure API to support dynamic indexes, following the same pattern as the existing configure.dynamic function.

Changes:

  • Added reconfigure.dynamic function to enable runtime reconfiguration of dynamic vector indexes
  • Fixed typo in type name from VectorIndexConfigDymamicUpdate to VectorIndexConfigDynamicUpdate with backward-compatible deprecated alias
  • Refactored existing reconfigure functions (flat, hfresh, hnsw) to use typed update objects instead of inline parameters

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/collections/configure/types/vectorIndex.ts Fixed typo in type name, added deprecated alias for backward compatibility, and updated type union and mapping references
src/collections/configure/vectorIndex.ts Added VectorIndexConfigDynamicUpdate import, refactored reconfigure.flat/hfresh/hnsw signatures, and implemented new reconfigure.dynamic function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bevzzz bevzzz requested a review from a team as a code owner February 27, 2026 09:43
@bevzzz bevzzz merged commit c6f1e01 into main Feb 27, 2026
19 checks passed
@bevzzz bevzzz deleted the fix/reconfigure-dynamic branch February 27, 2026 09:51
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.

3 participants