>From Ritik Raj <[email protected]>: Ritik Raj has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20008 )
Change subject: [ASTERIXDB-3601][STO] Removing warn log ...................................................................... [ASTERIXDB-3601][STO] Removing warn log - user model changes: no - storage format changes: no - interface changes: no Details: In ColumnRanges, there can be case where columnOffset is zero, if the columns are present in the pageZerosegment which is not being projected. Ext-ref: MB-66306 Change-Id: Iee50eb4d305fa6d2bab0cbc33f3b859e4f316605 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20008 Integration-Tests: Jenkins <[email protected]> Reviewed-by: Peeyush Gupta <[email protected]> Tested-by: Jenkins <[email protected]> --- M hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java 1 file changed, 27 insertions(+), 4 deletions(-) Approvals: Peeyush Gupta: Looks good to me, approved Jenkins: Verified; Verified Anon. E. Moose #1000171: diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java index de2d1e8..6cf43bf 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree-column/src/main/java/org/apache/hyracks/storage/am/lsm/btree/column/cloud/ColumnRanges.java @@ -126,10 +126,9 @@ int columnOrdinal = 0; for (int i = 0; i < numberOfColumns; i++) { - if (offsetColumnIndexPairs[i] == 0) { // any column's offset can't be zero - LOGGER.warn( - "Unexpected zero column offset at index {}. This may indicate a logic error or data inconsistency.", - i); + if (offsetColumnIndexPairs[i] == 0) { + //Any requested column's offset can't be zero + //In case a column is not being present in the accessed pageZero segments, it will be defaulted to 0 continue; } int columnIndex = getColumnIndexFromPair(offsetColumnIndexPairs[i]); -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20008 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: Iee50eb4d305fa6d2bab0cbc33f3b859e4f316605 Gerrit-Change-Number: 20008 Gerrit-PatchSet: 3 Gerrit-Owner: Ritik Raj <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Peeyush Gupta <[email protected]> Gerrit-Reviewer: Ritik Raj <[email protected]> Gerrit-MessageType: merged
