>From Hongyu Shi <[email protected]>:

Attention is currently required from: Hongyu Shi.

Hello Anon. E. Moose #1000171, Jenkins,

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21654?usp=email

to look at the new patch set (#3).

The following approvals got outdated and were removed:
Contrib-2 by Anon. E. Moose #1000171, Integration-Tests+1 by Jenkins, 
Verified-1 by Jenkins


Change subject: [ASTERIXDB-3817][COMP][RT] Filter unusable vectors before the 
build
......................................................................

[ASTERIXDB-3817][COMP][RT] Filter unusable vectors before the build

- user model changes: yes
- storage format changes: no
- interface changes: no

Details:
Apply isvector(field, dimension) as a StreamSelect ahead of the stages that read
the field in three jobs of vtree index build, and clean the scattered runtime
validation each job carried. The predicate is the one a user can run, so the
rows a vector index contains are exactly the rows for which isvector is true,
and a partial index is explainable with
SELECT COUNT(*) FROM ds WHERE NOT isvector(ds.emb, 384). The build error no
longer names the observed dimension, pointing at that query instead.

Ext-ref: MB-73665, MB-73439, MB-73300

Co-Authored-By: Claude Opus 5 <[email protected]>
Change-Id: I9133ac0dbf5a261bc4adbd8d382b84f44ac122b2
---
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/VectorQueries.xml
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/vector-distance-empty-vector/vector-distance-empty-vector.01.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/vector-distance-empty-vector/vector-distance-empty-vector.02.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/vector-distance-empty-vector/vector-distance-empty-vector.03.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/vector-distance-empty-vector/vector-distance-empty-vector.99.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/vector/vector-distance-empty-vector/vector-distance-empty-vector.03.adm
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/SecondaryVectorOperationsHelper.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/HierarchicalKMeansPlusPlusCentroidsOperatorDescriptor.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/VectorComponentExtractorOperatorDescriptor.java
M 
asterixdb/asterix-runtime/src/test/java/org/apache/asterix/runtime/operators/HierarchicalKMeansPlusPlusCentroidsOperatorDescriptorTest.java
M 
asterixdb/asterix-runtime/src/test/java/org/apache/asterix/runtime/operators/VectorComponentExtractorOperatorDescriptorTest.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-vtree/src/main/java/org/apache/hyracks/storage/am/lsm/vector/dataflow/QuantizedIndexCreateOperatorDescriptor.java
12 files changed, 255 insertions(+), 241 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/54/21654/3
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21654?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I9133ac0dbf5a261bc4adbd8d382b84f44ac122b2
Gerrit-Change-Number: 21654
Gerrit-PatchSet: 3
Gerrit-Owner: Hongyu Shi <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Hongyu Shi <[email protected]>

Reply via email to