>From Hongyu Shi <[email protected]>:

Attention is currently required from: Ali Alsuliman.

Hello Ali Alsuliman, Anon. E. Moose #1000171, Jenkins,

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

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

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

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


Change subject: [ASTERIXDB-3804][COMP] Drop the kmultiplier hint
......................................................................

[ASTERIXDB-3804][COMP] Drop the kmultiplier hint

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

Details: compiler.vector.kmultiplier duplicated the ann_distance 5th
argument and took precedence over it whenever both were set. Drop the
hint, its settable-option allowlist entry and the kMultiplier field it
fed on the VTree search operator so the per-query argument is the only
channel; the movie and glove tests pass 30 as that argument in place of
the SET. The trainseed half of this change is dropped: 21621 makes the
seed a WITH parameter, and this is rebased onto it.

Ext-ref: MB-72975
Change-Id: I67d27b8e74f5ee5e8eccc0d799d61ad45854a5ce
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/create-index-vtree-glove/create-index-vtree-glove.5.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/vector/create-index-vtree-movie/create-index-vtree-movie.5.query.sqlpp
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-vtree/src/main/java/org/apache/hyracks/storage/am/lsm/vector/dataflow/VTreeSearchOperatorDescriptor.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-vtree/src/main/java/org/apache/hyracks/storage/am/lsm/vector/dataflow/VTreeSearchOperatorNodePushable.java
7 files changed, 12 insertions(+), 37 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/02/21502/9
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21502?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: I67d27b8e74f5ee5e8eccc0d799d61ad45854a5ce
Gerrit-Change-Number: 21502
Gerrit-PatchSet: 9
Gerrit-Owner: Hongyu Shi <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>

Reply via email to