Murtadha Hubail has submitted this change and it was merged. Change subject: [ASTERIXDB-1974][STO] Re-interrupt Thread on Interrupt in PageQueue ......................................................................
[ASTERIXDB-1974][STO] Re-interrupt Thread on Interrupt in PageQueue - user model changes: no - storage format changes: no - interface changes: no Details: - Re-interrupt thread and throw exception when a thread adding a page to the PageQueue is interrupted. Change-Id: Ia71e6623315af8eb1e9e8c58608e01ff4eb99b40 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2128 Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Reviewed-by: Ian Maxon <ima...@apache.org> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Contrib: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu> --- M hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/AsyncFIFOPageQueueManager.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins: Verified; No violations found; ; Verified Ian Maxon: Looks good to me, approved diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/AsyncFIFOPageQueueManager.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/AsyncFIFOPageQueueManager.java index b693c91..d4c1a58 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/AsyncFIFOPageQueueManager.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/AsyncFIFOPageQueueManager.java @@ -66,8 +66,8 @@ throw new HyracksDataException("Queue is closing"); } } catch (InterruptedException e) { - // TODO Auto-generated catch block - e.printStackTrace(); + Thread.currentThread().interrupt(); + throw HyracksDataException.create(e); } } } -- To view, visit https://asterix-gerrit.ics.uci.edu/2128 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia71e6623315af8eb1e9e8c58608e01ff4eb99b40 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <mhub...@apache.org> Gerrit-Reviewer: Ian Maxon <ima...@apache.org> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Murtadha Hubail <mhub...@apache.org>
