This is an automated email from the ASF dual-hosted git repository. mblow pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit 3c796693ffa4ef2f6a51f27974ed957268d24256 Author: Michael Blow <[email protected]> AuthorDate: Tue May 27 16:31:27 2025 -0400 [NO ISSUE][STO][MISC] Fixes to handling of interrupted actions - Avoid double unpin on interrupt during unwind of pipeline by clearing the pinned page after successful unpin. - Propagate the interrupted exception when feasible when InvokeUtil helpers are used- previously these helpers would only set the interrupted flag on the current thread which can mask interrupted actions. (cherry picked from commit 65b6a27d1) Ext-ref: MB-67050 Change-Id: I2f8c64fa76beecbebcb40cf2a86866189cb1c55a Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19883 Tested-by: Jenkins <[email protected]> Reviewed-by: Ali Alsuliman <[email protected]> Reviewed-by: Michael Blow <[email protected]> --- .../common/context/GlobalVirtualBufferCache.java | 2 +- .../org/apache/hyracks/api/util/InvokeUtil.java | 150 +++++++++++++++++++-- .../am/btree/impls/BTreeCountingSearchCursor.java | 4 +- .../am/btree/impls/BTreeRangeSearchCursor.java | 17 +-- .../hyracks/storage/am/btree/impls/DiskBTree.java | 2 +- .../am/btree/impls/DiskBTreePointSearchCursor.java | 1 - .../am/btree/impls/DiskBTreeRangeSearchCursor.java | 3 +- .../common/impls/TreeIndexDiskOrderScanCursor.java | 5 +- .../impls/MultitenantVirtualBufferCache.java | 2 +- .../am/lsm/common/impls/VirtualBufferCache.java | 2 +- .../impls/AbstractOnDiskInvertedListCursor.java | 1 + .../FixedSizeElementInvertedListScanCursor.java | 2 +- .../storage/common/buffercache/BufferCache.java | 4 +- .../common/buffercache/DebugBufferCache.java | 2 +- .../storage/common/buffercache/IBufferCache.java | 3 +- .../storage/am/btree/StorageFileAccessTest.java | 38 +++--- .../am/lsm/btree/impl/TestVirtualBufferCache.java | 2 +- 17 files changed, 185 insertions(+), 55 deletions(-) diff --git a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/GlobalVirtualBufferCache.java b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/GlobalVirtualBufferCache.java index 156573013c..f8771430e4 100644 --- a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/GlobalVirtualBufferCache.java +++ b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/GlobalVirtualBufferCache.java @@ -313,7 +313,7 @@ public class GlobalVirtualBufferCache implements IVirtualBufferCache, ILifeCycle } @Override - public void unpin(ICachedPage page) throws HyracksDataException { + public void unpin(ICachedPage page) { vbc.unpin(page); } diff --git a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java index d331ab2135..a680ee11b3 100644 --- a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java +++ b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java @@ -35,6 +35,7 @@ import org.apache.hyracks.util.InterruptibleAction; import org.apache.hyracks.util.InterruptibleSupplier; import org.apache.hyracks.util.Span; import org.apache.hyracks.util.ThrowingAction; +import org.apache.hyracks.util.ThrowingConsumer; import org.apache.logging.log4j.Level; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; @@ -53,7 +54,12 @@ public class InvokeUtil { /** * Executes the passed interruptible, retrying if the operation is interrupted. Once the interruptible * completes, the current thread will be re-interrupted, if the original operation was interrupted. + * + * @deprecated this method does not throw an exception when the action is interrupted, or when the thread + * was interrupted prior to calling this method. This can lead to confusing behavior, if the caller + * does not check for interrupted thread state after calling this method. */ + @Deprecated public static void doUninterruptibly(InterruptibleAction interruptible) { boolean interrupted = Thread.interrupted(); try { @@ -211,10 +217,37 @@ public class InvokeUtil { } } - @SuppressWarnings({ "squid:S1181", "squid:S1193", "ConstantConditions" }) // catching Throwable, instanceofs + @SuppressWarnings({ "squid:S1181", "squid:S1193", "ConstantConditions", "UnreachableCode" }) + // catching Throwable, instanceofs, false-positive unreachable code + public static void tryWithCleanupsAsHyracks(ThrowingAction action, ThrowingAction... cleanups) + throws HyracksDataException { + Throwable savedT = null; + try { + action.run(); + } catch (Throwable t) { + savedT = t; + } finally { + for (ThrowingAction cleanup : cleanups) { + try { + cleanup.run(); + } catch (Throwable t) { + savedT = ExceptionUtils.suppress(savedT, t); + } + } + } + if (Thread.interrupted()) { + savedT = ExceptionUtils.suppress(savedT, new InterruptedException()); + } + if (savedT == null) { + return; + } + throw HyracksDataException.create(savedT); + } + + @SuppressWarnings({ "squid:S1181", "squid:S1193", "ConstantConditions", "UnreachableCode" }) + // catching Throwable, instanceofs, false-positive unreachable code public static void tryWithCleanups(ThrowingAction action, ThrowingAction... cleanups) throws Exception { Throwable savedT = null; - boolean suppressedInterrupted = false; try { action.run(); } catch (Throwable t) { @@ -223,6 +256,111 @@ public class InvokeUtil { for (ThrowingAction cleanup : cleanups) { try { cleanup.run(); + } catch (Throwable t) { + savedT = ExceptionUtils.suppress(savedT, t); + } + } + } + if (Thread.interrupted()) { + savedT = ExceptionUtils.suppress(savedT, new InterruptedException()); + } + if (savedT == null) { + return; + } + if (savedT instanceof Error) { + throw (Error) savedT; + } else if (savedT instanceof Exception) { + throw (Exception) savedT; + } else { + throw HyracksDataException.create(savedT); + } + } + + /** + * Runs the supplied action, and any specified cleanups. Any pending interruption will be cleared prior + * to running the action and all cleanups. An error will be logged if the action and/or any of the cleanups + * are themselves interrupted. Finally, if any action or cleanup was interrupted, or if the there was an + * interrupt cleared as part of running any of these activities, either an InterruptedException will be returned + * if the action & cleanups all ran without exception, or an InterruptedException will be suppressed into the + * exception if the action or any of the cleanups threw an exception. In the case where InterruptedException is + * suppressed, the current thread will be interrupted. + * + * @param action the action to run + * @param cleanups the cleanups to run after the action + * @return Exception if the action throws an exception or the action or any of the cleanups are interrupted, or if + * the current thread was interrupted before running the action or any of the cleanups. + */ + @SuppressWarnings({ "squid:S1181", "squid:S1193", "ConstantConditions", "UnreachableCode" }) + // catching Throwable, instanceofs, false-positive unreachable code + public static Exception tryUninterruptibleWithCleanups(Exception root, ThrowingAction action, + ThrowingAction... cleanups) { + try { + tryUninterruptibleWithCleanups(action, cleanups); + } catch (Exception e) { + root = ExceptionUtils.suppress(root, e); + } + return root; + } + + /** + * Runs the supplied action, and any specified cleanups. Any pending interruption will be cleared prior + * to running the action and all cleanups. An error will be logged if the action and/or any of the cleanups + * are themselves interrupted. Finally, if any action or cleanup was interrupted, or if the there was an + * interrupt cleared as part of running any of these activities, either an InterruptedException will be thrown + * if the action & cleanups all ran without exception, or an InterruptedException will be suppressed into the + * exception if the action or any of the cleanups threw an exception. In the case where InterruptedException is + * suppressed, the current thread will be interrupted. + * + * @param action the action to run + * @param cleanups the cleanups to run after the action + * @throws Exception if the action throws an exception or the action or any of the cleanups are interrupted. + */ + @SuppressWarnings({ "squid:S1181", "squid:S1193", "ConstantConditions", "UnreachableCode" }) + // catching Throwable, instanceofs, false-positive unreachable code + public static void tryUninterruptibleWithCleanups(ThrowingAction action, ThrowingAction... cleanups) + throws Exception { + Throwable savedT = null; + try { + runUninterruptible(action); + } catch (Throwable t) { + savedT = t; + } finally { + for (ThrowingAction cleanup : cleanups) { + try { + runUninterruptible(cleanup); + } catch (Throwable t) { + savedT = ExceptionUtils.suppress(savedT, t); + } + } + } + if (Thread.interrupted()) { + savedT = ExceptionUtils.suppress(savedT, new InterruptedException()); + } + if (savedT == null) { + return; + } + if (savedT instanceof Error) { + throw (Error) savedT; + } else if (savedT instanceof Exception) { + throw (Exception) savedT; + } else { + throw HyracksDataException.create(savedT); + } + } + + // catching Throwable, instanceofs, false-positive unreachable code + public static void tryWithCleanups(ThrowingAction action, ThrowingConsumer<Throwable>... cleanups) + throws Exception { + Throwable savedT = null; + boolean suppressedInterrupted = false; + try { + action.run(); + } catch (Throwable t) { + savedT = t; + } finally { + for (ThrowingConsumer cleanup : cleanups) { + try { + cleanup.process(savedT); } catch (Throwable t) { if (savedT != null) { savedT.addSuppressed(t); @@ -329,16 +467,12 @@ public class InvokeUtil { boolean interrupted = Thread.interrupted(); try { action.run(); - if (Thread.interrupted()) { + if (interrupted || Thread.interrupted()) { throw new InterruptedException(); } } catch (InterruptedException e) { LOGGER.error("uninterruptible action {} was interrupted!", action, e); - interrupted = true; - } finally { - if (interrupted) { - Thread.currentThread().interrupt(); - } + throw e; } } diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeCountingSearchCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeCountingSearchCursor.java index 71df593056..39e2a38030 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeCountingSearchCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeCountingSearchCursor.java @@ -108,13 +108,14 @@ public class BTreeCountingSearchCursor extends EnforcedIndexCursor implements IT stopTupleIndex = getHighKeyIndex(); } - private void releasePage() throws HyracksDataException { + private void releasePage() { if (exclusiveLatchNodes) { page.releaseWriteLatch(isPageDirty); } else { page.releaseReadLatch(); } bufferCache.unpin(page); + page = null; } private void fetchNextLeafPage(int nextLeafPage) throws HyracksDataException { @@ -218,7 +219,6 @@ public class BTreeCountingSearchCursor extends EnforcedIndexCursor implements IT } tupleBuilder.reset(); tupleIndex = 0; - page = null; isPageDirty = false; pred = null; count = -1; diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeRangeSearchCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeRangeSearchCursor.java index cf3727a754..99e2d0bcaa 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeRangeSearchCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/BTreeRangeSearchCursor.java @@ -46,7 +46,6 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree protected final IBTreeLeafFrame frame; protected final ITreeIndexTupleReference frameTuple; protected final boolean exclusiveLatchNodes; - protected boolean isPageDirty; protected IBufferCache bufferCache = null; protected int fileId = -1; @@ -118,7 +117,6 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree releasePage(); } page = nextLeaf; - isPageDirty = false; frame.setPage(page); pageId = nextLeafPage; nextLeafPage = frame.getNextLeaf(); @@ -159,8 +157,6 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree reconciliationTuple.reset(tupleBuilder.getFieldEndOffsets(), tupleBuilder.getByteArray()); releasePage(); - page = null; - isPageDirty = false; // reconcile searchCb.reconcile(reconciliationTuple); @@ -236,7 +232,6 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree originalKeyCmp = initialState.getOriginalKeyComparator(); pageId = ((BTreeCursorInitialState) initialState).getPageId(); page = initialState.getPage(); - isPageDirty = false; frame.setPage(page); pred = (RangePredicate) searchPred; @@ -267,7 +262,7 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree stopTupleIndex = getHighKeyIndex(); } - protected void resetBeforeOpen() throws HyracksDataException { + protected void resetBeforeOpen() { releasePage(); } @@ -278,8 +273,6 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree } tupleIndex = 0; - page = null; - isPageDirty = false; pred = null; } @@ -298,13 +291,17 @@ public class BTreeRangeSearchCursor extends EnforcedIndexCursor implements ITree return exclusiveLatchNodes; } - protected void releasePage() throws HyracksDataException { + /** + * Releases the page and unpins it from the buffer cache, clearing the reference to the page. + */ + protected void releasePage() { if (exclusiveLatchNodes) { - page.releaseWriteLatch(isPageDirty); + page.releaseWriteLatch(false); } else { page.releaseReadLatch(); } bufferCache.unpin(page); + page = null; } protected ICachedPage acquirePage(int pageId) throws HyracksDataException { diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTree.java b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTree.java index 282aad9511..1cbe31dde1 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTree.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTree.java @@ -222,7 +222,7 @@ public class DiskBTree extends BTree { } @Override - protected void releasePage() throws HyracksDataException { + protected void releasePage() { if (page != null) { bufferCache.unpin(page); page = null; diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreePointSearchCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreePointSearchCursor.java index 1bf3ecfca6..a1f04c6ee2 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreePointSearchCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreePointSearchCursor.java @@ -69,7 +69,6 @@ public class DiskBTreePointSearchCursor extends DiskBTreeRangeSearchCursor { originalKeyCmp = initialState.getOriginalKeyComparator(); pageId = ((BTreeCursorInitialState) initialState).getPageId(); page = initialState.getPage(); - isPageDirty = false; frame.setPage(page); pred = (RangePredicate) searchPred; diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreeRangeSearchCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreeRangeSearchCursor.java index d788398480..d62ca913af 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreeRangeSearchCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/DiskBTreeRangeSearchCursor.java @@ -66,8 +66,9 @@ public class DiskBTreeRangeSearchCursor extends BTreeRangeSearchCursor { } @Override - protected void releasePage() throws HyracksDataException { + protected void releasePage() { bufferCache.unpin(page); + page = null; } @Override diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/impls/TreeIndexDiskOrderScanCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/impls/TreeIndexDiskOrderScanCursor.java index 36fba767aa..b0a2c71454 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/impls/TreeIndexDiskOrderScanCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/impls/TreeIndexDiskOrderScanCursor.java @@ -142,7 +142,10 @@ public class TreeIndexDiskOrderScanCursor extends EnforcedIndexCursor implements return false; } - protected void releasePage() throws HyracksDataException { + /** + * Releases the current page, if it is not null, clearing the reference to it. + */ + protected void releasePage() { if (page != null) { page.releaseReadLatch(); bufferCache.unpin(page); diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/MultitenantVirtualBufferCache.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/MultitenantVirtualBufferCache.java index e77acea941..f801025a7b 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/MultitenantVirtualBufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/MultitenantVirtualBufferCache.java @@ -70,7 +70,7 @@ public class MultitenantVirtualBufferCache implements IVirtualBufferCache { } @Override - public void unpin(ICachedPage page) throws HyracksDataException { + public void unpin(ICachedPage page) { vbc.unpin(page); } diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java index 5871b31360..60b38dd4db 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java @@ -317,7 +317,7 @@ public class VirtualBufferCache implements IVirtualBufferCache { } @Override - public void unpin(ICachedPage page) throws HyracksDataException { + public void unpin(ICachedPage page) { } @Override diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/impls/AbstractOnDiskInvertedListCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/impls/AbstractOnDiskInvertedListCursor.java index 9886da469c..b68a1631e9 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/impls/AbstractOnDiskInvertedListCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/impls/AbstractOnDiskInvertedListCursor.java @@ -219,6 +219,7 @@ public abstract class AbstractOnDiskInvertedListCursor extends AbstractInvertedL currentBufferIdx++; bufferCache.unpin(page); + page = null; bufferEndPageId = i; // Buffer full? diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/fixedsize/FixedSizeElementInvertedListScanCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/fixedsize/FixedSizeElementInvertedListScanCursor.java index 458eb6bbcc..cdc0701303 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/fixedsize/FixedSizeElementInvertedListScanCursor.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/fixedsize/FixedSizeElementInvertedListScanCursor.java @@ -127,7 +127,7 @@ public class FixedSizeElementInvertedListScanCursor extends AbstractInvertedList } @Override - public void unloadPages() throws HyracksDataException { + public void unloadPages() { if (pinned) { bufferCache.unpin(page); pinned = false; diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/BufferCache.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/BufferCache.java index d7be126860..cf3c1043d6 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/BufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/BufferCache.java @@ -573,9 +573,9 @@ public class BufferCache implements IBufferCacheInternal, ILifeCycleComponent, I } @Override - public void unpin(ICachedPage page) throws HyracksDataException { + public void unpin(ICachedPage page) { if (closed) { - throw new HyracksDataException("unpin called on a closed cache"); + throw new IllegalStateException("unpin called on a closed cache"); } int pinCount = ((CachedPage) page).decrementAndGetPinCount(); if (DEBUG && pinCount == 0) { diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/DebugBufferCache.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/DebugBufferCache.java index 8c3d492415..88f3035503 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/DebugBufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/DebugBufferCache.java @@ -84,7 +84,7 @@ public class DebugBufferCache implements IBufferCache { } @Override - public void unpin(ICachedPage page) throws HyracksDataException { + public void unpin(ICachedPage page) { bufferCache.unpin(page); unpinCount.addAndGet(1); } diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IBufferCache.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IBufferCache.java index df0bea854a..f133d80bbe 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IBufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IBufferCache.java @@ -109,9 +109,8 @@ public interface IBufferCache { * * @param page * the page - * @throws HyracksDataException */ - void unpin(ICachedPage page) throws HyracksDataException; + void unpin(ICachedPage page); /** * Flush the page if it is dirty diff --git a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-btree-test/src/test/java/org/apache/hyracks/storage/am/btree/StorageFileAccessTest.java b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-btree-test/src/test/java/org/apache/hyracks/storage/am/btree/StorageFileAccessTest.java index 576c6bb6ea..226d3edc22 100644 --- a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-btree-test/src/test/java/org/apache/hyracks/storage/am/btree/StorageFileAccessTest.java +++ b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-btree-test/src/test/java/org/apache/hyracks/storage/am/btree/StorageFileAccessTest.java @@ -143,31 +143,27 @@ public class StorageFileAccessTest extends AbstractBTreeTest { private void unpinRandomPage() { int index = Math.abs(rnd.nextInt() % pinnedPages.size()); - try { - PinnedLatchedPage plPage = pinnedPages.get(index); + PinnedLatchedPage plPage = pinnedPages.get(index); - if (plPage.latch != null) { - if (plPage.latch == LatchType.LATCH_S) { - if (LOGGER.isInfoEnabled()) { - LOGGER.info(workerId + " S UNLATCHING: " + plPage.pageId); - } - plPage.page.releaseReadLatch(); - } else { - if (LOGGER.isInfoEnabled()) { - LOGGER.info(workerId + " X UNLATCHING: " + plPage.pageId); - } - plPage.page.releaseWriteLatch(true); + if (plPage.latch != null) { + if (plPage.latch == LatchType.LATCH_S) { + if (LOGGER.isInfoEnabled()) { + LOGGER.info(workerId + " S UNLATCHING: " + plPage.pageId); } + plPage.page.releaseReadLatch(); + } else { + if (LOGGER.isInfoEnabled()) { + LOGGER.info(workerId + " X UNLATCHING: " + plPage.pageId); + } + plPage.page.releaseWriteLatch(true); } - if (LOGGER.isInfoEnabled()) { - LOGGER.info(workerId + " UNPINNING PAGE: " + plPage.pageId); - } - - bufferCache.unpin(plPage.page); - pinnedPages.remove(index); - } catch (HyracksDataException e) { - e.printStackTrace(); } + if (LOGGER.isInfoEnabled()) { + LOGGER.info(workerId + " UNPINNING PAGE: " + plPage.pageId); + } + + bufferCache.unpin(plPage.page); + pinnedPages.remove(index); } private void openFile() { diff --git a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestVirtualBufferCache.java b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestVirtualBufferCache.java index 8cad497184..d7a83dce20 100644 --- a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestVirtualBufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestVirtualBufferCache.java @@ -101,7 +101,7 @@ public class TestVirtualBufferCache implements IVirtualBufferCache { } @Override - public void unpin(ICachedPage page) throws HyracksDataException { + public void unpin(ICachedPage page) { vbc.unpin(page); }
