Skip to content

Fixes around CU copies and delete deadlocks - #2082

Merged
kevin-montrose merged 3 commits into
mainfrom
users/kmontrose/vectorSessionFunctionSizingFixes
Aug 24, 2026
Merged

Fixes around CU copies and delete deadlocks#2082
kevin-montrose merged 3 commits into
mainfrom
users/kmontrose/vectorSessionFunctionSizingFixes

Conversation

@kevin-montrose

Copy link
Copy Markdown
Contributor

Two fixes:

  • making sure the VectorSessionFunctions.CopyUpdater doesn't copy extra data (which can result in a span length check failing)
  • remove blocking on marking deletion requests, this has been superseded by deletion logic during recovery and can lead to deadlocks

Copilot AI balanced review requested due to automatic review settings August 24, 2026 14:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes Vector Set copy updates and removes blocking deletion waits.

Changes:

  • Limits copy updates to the requested size.
  • Makes deletion marking asynchronous.
  • Updates deletion documentation.

Reviewed changes

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

File Description
website/docs/dev/vector-sets.md Updates deletion workflow docs.
VectorSessionFunctions.cs Bounds copy-update spans.
VectorManager.cs Removes synchronous cleanup wait.
VectorManager.Cleanup.cs Simplifies asynchronous cleanup requests.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread website/docs/dev/vector-sets.md
Comment thread libs/server/Resp/Vector/VectorManager.Cleanup.cs
Comment thread libs/server/Storage/Functions/VectorStore/VectorSessionFunctions.cs
Comment thread libs/server/Resp/Vector/VectorManager.Cleanup.cs
@kevin-montrose
kevin-montrose merged commit 1322207 into main Aug 24, 2026
170 checks passed
@kevin-montrose
kevin-montrose deleted the users/kmontrose/vectorSessionFunctionSizingFixes branch August 24, 2026 17:23
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