>From Ritik Raj <ritik....@couchbase.com>:

Attention is currently required from: Ritik Raj.
Hello Peeyush Gupta, Jenkins, Anon. E. Moose #1000171,

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

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20127

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

Change subject: [ASTERIXDB-3601][STO] Fix calculation of presentColumns
......................................................................

[ASTERIXDB-3601][STO] Fix calculation of presentColumns

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

Details:
During the merge, there was an issue setting
up the columnIndexes from the additional pageSegments,
which caused the extra columns to be ignored

Ext-ref: MB-67741
Change-Id: Id43277dc678ab6578c27d3cf114063acbcc849fb
---
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/column/io/merge/ASTERIXDB-3601/ASTERIXDB-3601.003.adm
M 
asterixdb/asterix-column/src/main/java/org/apache/asterix/column/zero/writers/multipage/SparseColumnMultiPageZeroReader.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/column/io/merge/ASTERIXDB-3601/ASTERIXDB-3601.001.ddl.sqlpp
M 
asterixdb/asterix-column/src/main/java/org/apache/asterix/column/tuple/MergeColumnTupleReference.java
M 
asterixdb/asterix-column/src/main/java/org/apache/asterix/column/zero/readers/DefaultColumnPageZeroReader.java
M 
asterixdb/asterix-column/src/main/java/org/apache/asterix/column/zero/readers/SparseColumnPageZeroReader.java
M 
asterixdb/asterix-column/src/main/java/org/apache/asterix/column/zero/writers/multipage/DefaultColumnMultiPageZeroReader.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/column/io/merge/ASTERIXDB-3601/ASTERIXDB-3601.002.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/column/io/merge/ASTERIXDB-3601/ASTERIXDB-3601.003.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/testsuite_single_partition_sqlpp.xml
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/impls/btree/IColumnPageZeroReader.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/impls/btree/ColumnBTreeReadLeafFrame.java
12 files changed, 1,976 insertions(+), 12 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/27/20127/5
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20127
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: ionic
Gerrit-Change-Id: Id43277dc678ab6578c27d3cf114063acbcc849fb
Gerrit-Change-Number: 20127
Gerrit-PatchSet: 5
Gerrit-Owner: Ritik Raj <ritik....@couchbase.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Peeyush Gupta <peeyush.gu...@couchbase.com>
Gerrit-Attention: Ritik Raj <ritik....@couchbase.com>
Gerrit-MessageType: newpatchset

Reply via email to