validator: add Alternator FLOAT32VECTOR type end-to-end tests#437
validator: add Alternator FLOAT32VECTOR type end-to-end tests#437QuerthDP wants to merge 1 commit into
Conversation
b9b620c to
7952f36
Compare
fdb2975 to
4dae248
Compare
m-szymon
left a comment
There was a problem hiding this comment.
- I was considering adding such tests to types.rs tests (added here #467). I deliberately named it types, not key_types.
- Adding optimized data to index and querying are slightly separate issues.
- I was considering defining test datasets using "binary" type and only extending the json injector to convert them to FLOAT32VECTOR when expected - it could better reuse existing utils.
But it works, so consider if you would like to improve anything and we can merge it. I can later refactor.
Add end-to-end coverage for the Alternator optimized vector type using requests with the new encoding injected into the JSON body. Verify that queries work for items stored with the optimized vector representation and remain compatible with the existing L-based vector representation in the same index.
4dae248 to
97c9968
Compare
|
Changelog:
|
|
It is easier to ask AI to do it, than explain it: 3459c12 The problem is the I also noticed that other test in that file don't actually check returned values. With that fixed, we could extend and use the |
@m-szymon Does it mean that something needs to be done in this PR? |
Yes - one small inconsistency that |
Add end-to-end coverage for the Alternator optimized vector type using requests with the new encoding injected into the JSON body.
Verify that queries work for items stored with the optimized vector representation and remain compatible with the existing L-based vector representation in the same index.
Fixes: VECTOR-666