Skip to content

feat(serverless): add processing time to CollectionsService responses - #497

Merged
generall merged 1 commit into
mainfrom
feat/serverless-collections-response-time
Sep 21, 2026
Merged

generall merged 1 commit into
mainfrom
feat/serverless-collections-response-time

Conversation

@qdrant-cloud-bot

Copy link
Copy Markdown
Contributor

Summary

  • Add double time (with the standard “Time spent to process” comment) to all qdrant.serverless.CollectionsService response messages: CreateCollectionResponse, DeleteCollectionResponse, GetCollectionResponse, ListCollectionsResponse.
  • Regenerate Go / Python / TypeScript bindings.

Field numbers use the next free tag on each message (3 or 4) because existing fields already occupy tag 2, matching how Qdrant’s own ScrollResponse places time when needed.

Test plan

  • make generate / make lint clean
  • Downstream qdrant-serverless collection-manager fills time on success paths
  • Rust / Python clients pick up the new field after syncing this proto

Align tenant-facing collection RPCs with the Qdrant response convention by
adding a `time` field so clients can observe request latency.
@qdrant-cloud-bot
qdrant-cloud-bot requested a review from a team as a code owner September 18, 2026 17:59
@github-actions

Copy link
Copy Markdown

PR Packages Published

Python Package:

  • Version: 0.189.0.dev497+12f2399
  • Package: qdrant-cloud-public-api
  • Registry: https://us-python.pkg.dev/qdrant-cloud/python/
  • To update run: uv add qdrant-cloud-public-api==0.189.0.dev497+12f2399

NPM Package:

  • Version: 0.189.0-dev497.12f2399
  • Package: @qdrant/qdrant-cloud-public-api
  • Registry: https://us-npm.pkg.dev/qdrant-cloud/npm/
  • To update run: npm install @qdrant/qdrant-cloud-public-api@0.189.0-dev497.12f2399

@github-actions

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Pull Request / linting (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedSep 18, 2026, 6:00 PM

@qdrant-cloud-bot

Copy link
Copy Markdown
Contributor Author

Downstream follow-ups:

Python bindings for this contract are regenerated in this PR under gen/python/qdrant/serverless/ (there is no separate serverless surface in qdrant/qdrant-client yet).

@qdrant-cloud-bot

Copy link
Copy Markdown
Contributor Author

Also: Python client (dev) — https://github.com/qdrant/qdrant-client/pull/

@qdrant-cloud-bot

Copy link
Copy Markdown
Contributor Author

Also: Python client (dev) — qdrant/qdrant-client#1452

@@ -234,6 +236,8 @@ message DeleteCollectionResponse {
bool deleted = 1;
// Number of storage objects removed.
uint32 objects_deleted = 2;

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.

do we want to drop this (separate PR)?

@Robert-Stam Robert-Stam 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.

LGTM, see question (not blocking this PR)

@generall
generall merged commit 21204a5 into main Sep 21, 2026
11 checks passed
@generall
generall deleted the feat/serverless-collections-response-time branch September 21, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants