Skip to content

[v1.16.0] add into_vector interface for VectorOutput#237

Merged
timvisee merged 4 commits intodevfrom
interface-for-reading-vectors
Nov 12, 2025
Merged

[v1.16.0] add into_vector interface for VectorOutput#237
timvisee merged 4 commits intodevfrom
interface-for-reading-vectors

Conversation

@generall
Copy link
Member

@generall generall commented Aug 29, 2025

VectorOutput field contains deprecated vector representation of fields data, indices and vectors_count. We want to switch it to oneof vector field, which is more extendable and easier to work.

In order to do it nicely, we need to do it in several steps:

  • Step 1 (current, 1.16.0): Provide universal interface to read vectors, independent from protocol-level vector representation, deprecate direct access to fields.
  • Step 2: Switch server-side field from old to new (1.17.0)
  • Step 3: Remove old fields (1.18.0)

@generall generall requested a review from Anush008 August 29, 2025 20:04
Copy link
Member

@Anush008 Anush008 left a comment

Choose a reason for hiding this comment

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

@generall
Copy link
Member Author

Should we also mark deprecation of these fields too?

I think yes, we already use builders for those, so the transition should be even easier

@generall
Copy link
Member Author

It would depend on qdrant/qdrant#7185 unfortunately though

@generall generall changed the title [v1.16.0, do not merge] add into_vector interface for VectorOutput [v1.16.0] add into_vector interface for VectorOutput Nov 12, 2025
@timvisee timvisee force-pushed the interface-for-reading-vectors branch from f9dab20 to 38b4a87 Compare November 12, 2025 13:54
@timvisee timvisee changed the base branch from master to dev November 12, 2025 13:54
@timvisee timvisee merged commit 2a568c7 into dev Nov 12, 2025
2 checks passed
timvisee added a commit that referenced this pull request Nov 12, 2025
* add into_vector interface for VectorOutput

* fix clippy

* Also deprecated vector input fields

* Bump deprecation notice

---------

Co-authored-by: timvisee <tim@visee.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants