You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2025/07/14 16:37:15 UTC
Change in asterixdb[ionic]: [ASTERIXDB-3601][STO] Fixing Merge failure
From Ritik Raj <ri...@couchbase.com>:
Attention is currently required from: Ritik Raj.
Hello Jenkins, Anon. E. Moose #1000171,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20038
to look at the new patch set (#5).
Change subject: [ASTERIXDB-3601][STO] Fixing Merge failure
......................................................................
[ASTERIXDB-3601][STO] Fixing Merge failure
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
While bulkloading during merge, we calculate the present
columns in each of the leaf. But, there can be case where
a cursor gets closed because all the tuples have been read.
Closing a range cursor releases the page, hence can be reused.
While Bulkloading, even after the rangeCursor was closed, the
leaf was being asked for the present set of columns. Since, the
page has been reused, it contained differnt buffer, which when
read was giving wrong column details.
Hence, fixing this by calculating the info while reset happens
with new leaf, which always comes before closing the cursor.
Ext-ref: MB-67570
Change-Id: I87b3a084d01986dd5c2abd9452a2ad5619fbab15
---
M asterixdb/asterix-column/src/main/java/org/apache/asterix/column/zero/writers/multipage/SparseColumnMultiPageZeroReader.java
M hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java
M hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/buffercache/read/CloudColumnReadContext.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 hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/sweep/ColumnSweeper.java
M asterixdb/asterix-column/src/main/java/org/apache/asterix/column/zero/writers/multipage/DefaultColumnMultiPageZeroReader.java
7 files changed, 78 insertions(+), 21 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/38/20038/5
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20038
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: I87b3a084d01986dd5c2abd9452a2ad5619fbab15
Gerrit-Change-Number: 20038
Gerrit-PatchSet: 5
Gerrit-Owner: Ritik Raj <ri...@couchbase.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Attention: Ritik Raj <ri...@couchbase.com>
Gerrit-MessageType: newpatchset