Skip to content

chore(vendor): track zvec 8397071 - #23

Merged
chinaux merged 1 commit into
mainfrom
chore/vendor-sync-8397071
Oct 20, 2026
Merged

chinaux merged 1 commit into
mainfrom
chore/vendor-sync-8397071

Conversation

@chinaux

@chinaux chinaux commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

Vendor bump to upstream main HEAD, one commit ahead of 89ba922 (#22).

vendor/zvec 89ba922 → 8397071:

  • 8397071 fix(sqlengine): keep the exact FLOAT and DOUBLE bound in a forward filter (#818) — touches src/db/sqlengine/analyzer/query_info_helper.cc (+16/-2) and adds tests/db/collection_float_range_filter_test.cc. C++ only.

No C-API change: git diff --stat 89ba922..8397071 -- src/include/zvec/c_api.h src/include/zvec/export.h src/binding/c is empty, so no Rust adaptation is needed.

Relevant to us because it fixes inclusive/exclusive FLOAT and DOUBLE bounds in a forward filter — i.e. the scalar range-filter path that zvec-rust's SearchQuery filters and scalar_query_test / string_filter_test exercise.

Validation

  • PR CI builds the bumped submodule from source on all five platforms and runs the full test suite (cargo test --workspace --tests).
  • build-prebuilt.yml is not triggered by PRs, so it is dispatched manually on this branch for all eight targets (link below).
  • Will merge only when both are green.

One commit ahead of 89ba922:

- 8397071 fix(sqlengine): keep the exact FLOAT and DOUBLE bound in a
  forward filter (#818)

C++ only (query_info_helper.cc plus a new float range filter test);
c_api.h, export.h and src/binding/c/ are identical at both revisions,
so no Rust adaptation is needed.
@chinaux
chinaux merged commit ec7e4bb into main Oct 20, 2026
17 checks passed
@chinaux
chinaux deleted the chore/vendor-sync-8397071 branch October 20, 2026 12:41
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.

1 participant