Repository navigation
Conversation
The already-exists and not-found checks also matched a ValueError by its message text. Only qdrant-client's local mode raises those, and the store no longer runs in local mode: against a server, the client raises UnexpectedResponse with an HTTP status over REST and AioRpcError with a gRPC status code over gRPC. The message match and the ValueError the preparation and purge caught for it go. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
edwinyyyu
added
horizontal scaling
This was referenced Oct 7, 2026
Draft
[session storage 2/2] Remove open-or-create from both stores, and close from the segment store
#1625
Draft
[qdrant options] Let a deployment tune a Qdrant collection's HNSW, optimizers and quantization
#1618
Draft
edwinyyyu
merged commit Oct 7, 2026
dc53da8
into
MemMachine:feat/horizontal-scaling
39 checks passed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Purpose of the change
Summary
The Qdrant store's already-exists and not-found checks also matched a
ValueErrorby its message text, and its collection preparation, payload index creation, and purge caughtValueErrorfor them. Only qdrant-client's local mode raises those errors, and the store runs only against a server: over REST the client raisesUnexpectedResponsewith an HTTP status, and over gRPCAioRpcErrorwith a status code. The message match and theValueErrorcatches go; both checks classify by status code alone.Commits
Stack
22 open PRs: three independent PRs, and the vector store tree of short parallel branches. #1736's, #1779's, and #1788's GitHub base is
feat/horizontal-scalingand the others' ismain, since the branches are in a fork and a pull request can target only this repository's branches; the on column gives the order the PRs build on each other instead. A stacked PR's diff on GitHub includes the PRs under it until they merge.Independent of the vector store tree, directly on
main:mainmainmainThe vector store tree. Each PR builds on the one in its on column; PRs on the same parent are parallel branches and do not depend on each other. #1631 is closed, superseded by #1733–#1736, which hold its changes split in four, with review changes since. #1670 and #1702 sit beneath #1627, whose code depends on them. Until the PRs under it merge, their changes show in a stacked PR's diff.
mainfeat/horizontal-scaling)feat/horizontal-scalingfeat/horizontal-scaling)feat/horizontal-scalingfeat/horizontal-scaling)feat/horizontal-scalingfeat/horizontal-scalingfeat/horizontal-scaling)feat/horizontal-scalingfeat/horizontal-scalingThis PR is its one commit,
8f69592db, directly onfeat/horizontal-scaling.Verification
At the head:
ruff check,ruff format --check, andty checkare clean; the full server suite without integration tests passes (2181 passed, 8 skipped); in test containers, the Qdrant store's and the resource manager's integration tests pass against Qdrant 1.19.1 and PostgreSQL (211 passed), over REST and gRPC, the racing creation tests among them. In qdrant-client 1.19.0, outside local mode, only the unused migration helper raises aValueErrornaming an existing or missing resource.🤖 Generated with Claude Code