Michael Blow has submitted this change and it was merged. Change subject: [ASTERIXDB-2256] Hand-prepare two problematic files for source format ......................................................................
[ASTERIXDB-2256] Hand-prepare two problematic files for source format Change-Id: Ibcb24e807979629f25892d58bff1c44c061b42e3 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2334 Reviewed-by: Murtadha Hubail <[email protected]> Integration-Tests: Murtadha Hubail <[email protected]> Tested-by: Murtadha Hubail <[email protected]> --- M hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/search/SearchResult.java M hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/MathUtil.java 2 files changed, 5 insertions(+), 44 deletions(-) Approvals: Murtadha Hubail: Looks good to me, approved; Verified; Verified diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/search/SearchResult.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/search/SearchResult.java index 37eb600..2da1434 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/search/SearchResult.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/search/SearchResult.java @@ -143,47 +143,4 @@ return numResults; } - // TODO: This code may help to clean up the core list-merging algorithms. - /* - public SearchResultCursor getCursor() { - cursor.reset(); - return cursor; - } - - public class SearchResultCursor { - private int bufferIndex; - private int resultIndex; - private int frameResultIndex; - private ByteBuffer currentBuffer; - - public void reset() { - bufferIndex = 0; - resultIndex = 0; - frameResultIndex = 0; - currentBuffer = buffers.get(0); - resultFrameTupleAcc.reset(currentBuffer); - } - - public boolean hasNext() { - return resultIndex < numResults; - } - - public void next() { - resultTuple.reset(currentBuffer.array(), resultFrameTupleAcc.getTupleStartOffset(frameResultIndex)); - if (frameResultIndex < resultFrameTupleAcc.getTupleCount()) { - frameResultIndex++; - } else { - bufferIndex++; - currentBuffer = buffers.get(bufferIndex); - resultFrameTupleAcc.reset(currentBuffer); - frameResultIndex = 0; - } - resultIndex++; - } - - public ITupleReference getTuple() { - return resultTuple; - } - } - */ } diff --git a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/MathUtil.java b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/MathUtil.java index 2b157e4..2581934 100644 --- a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/MathUtil.java +++ b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/MathUtil.java @@ -20,6 +20,10 @@ package org.apache.hyracks.util; public class MathUtil { + + private MathUtil() { + } + /** * Fast way to calculate the log2(x). Note: x should be >= 1. * @@ -44,7 +48,7 @@ log |= 4; } - if (n > 0b11) { + if (n > 3) { n >>>= 2; log |= 2; } -- To view, visit https://asterix-gerrit.ics.uci.edu/2334 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibcb24e807979629f25892d58bff1c44c061b42e3 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]>
