Ian Maxon has submitted this change and it was merged. Change subject: [ASTERIXDB-2183][STO] Remove On-disk Inverted List Latch ......................................................................
[ASTERIXDB-2183][STO] Remove On-disk Inverted List Latch Change-Id: Idcf0df5119824f0ef6761d514ccbd82af678a797 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2328 Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Contrib: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Reviewed-by: Ian Maxon <ima...@apache.org> --- M hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/FixedSizeElementInvertedListCursor.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Anon. E. Moose #1000171: Jenkins: Verified; No violations found; ; Verified Ian Maxon: Looks good to me, approved diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/FixedSizeElementInvertedListCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/FixedSizeElementInvertedListCursor.java index 94ce348..75bdbc4 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/FixedSizeElementInvertedListCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/FixedSizeElementInvertedListCursor.java @@ -96,7 +96,6 @@ int pix = 0; for (int i = startPageId; i <= endPageId; i++) { pages[pix] = bufferCache.pin(BufferedFileHandle.getDiskPageId(fileId, i), false); - pages[pix].acquireReadLatch(); pix++; } pinned = true; @@ -106,7 +105,6 @@ public void unpinPages() throws HyracksDataException { int numPages = endPageId - startPageId + 1; for (int i = 0; i < numPages; i++) { - pages[i].releaseReadLatch(); bufferCache.unpin(pages[i]); } pinned = false; -- To view, visit https://asterix-gerrit.ics.uci.edu/2328 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idcf0df5119824f0ef6761d514ccbd82af678a797 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon <ima...@apache.org> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon <ima...@apache.org> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Taewoo Kim <taew...@uci.edu> Gerrit-Reviewer: Taewoo Kim <wangs...@gmail.com>
