>From Ritik Raj <[email protected]>: Attention is currently required from: Peeyush Gupta. Ritik Raj has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20151 )
Change subject: [NO ISSUE][STO] Fixed multiple issue ...................................................................... Patch Set 6: (1 comment) File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20151/comment/b9c37250_2579eaa6 PS6, Line 266: Arrays.fill(offsetColumnIndexPairs, 0, numberOfColumns + 1, Long.MAX_VALUE) > Why do we need this? Is this to do with sorting offsetColumnIndexPairs? yes, in case of sparse columns, I believe there might be a case, where the number of columns to be considered is X, while the length of the offsetColumnIndexPairs is Y, where Y > X, in that case for range [X+1..Y) there might be values smaller than what we need, which on sorting will come forward. Causing to read a columnIndex which does not exist in the current leaf. -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20151 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: I1a5725e36301537a0926781d1d875ff62d523545 Gerrit-Change-Number: 20151 Gerrit-PatchSet: 6 Gerrit-Owner: Ritik Raj <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Peeyush Gupta <[email protected]> Gerrit-Attention: Peeyush Gupta <[email protected]> Gerrit-Comment-Date: Mon, 11 Aug 2025 04:04:18 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Peeyush Gupta <[email protected]> Gerrit-MessageType: comment
