>From Ritik Raj <[email protected]>:
Attention is currently required from: Ritik Raj.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20967?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified-1 by Jenkins
Change subject: [NO ISSUE][HYR] Fix NPE in AbstractIoOperation and improve
cloud interrupt handling
......................................................................
[NO ISSUE][HYR] Fix NPE in AbstractIoOperation and improve cloud interrupt
handling
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
During a distributed job cancellation (e.g., HYR0034), the Hyracks
framework interrupts active threads. If a thread is interrupted
during an LSM bulk load, it enters an abort sequence that triggers
AbstractIoOperation.cleanup().
Because the interrupt originated externally, the I/O operation never
registered an internal failure. This resulted in getFailure() returning
null, causing a NullPointerException when cleanup() attempted to attach
suppressed teardown exceptions. This NPE masked the true state of the system.
Furthermore, CloudRetryableRequestUtil failed to recognize the AWS SDK's
AbortedException as a standard interrupt, causing it to violently rethrow
the exception instead of cleanly suppressing it during cloud teardowns.
Change-Id: If36c1336f8e8377f2d410c8878d168f1209bcf87
---
M
hyracks-fullstack/hyracks/hyracks-cloud/src/main/java/org/apache/hyracks/cloud/util/CloudRetryableRequestUtil.java
M
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/AbstractIoOperation.java
M
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/LSMIndexDiskComponentBulkLoader.java
M
hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/BufferCache.java
4 files changed, 19 insertions(+), 4 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/67/20967/4
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20967?usp=email
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: If36c1336f8e8377f2d410c8878d168f1209bcf87
Gerrit-Change-Number: 20967
Gerrit-PatchSet: 4
Gerrit-Owner: Ritik Raj <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-CC: Anon. E. Moose #1000171
Gerrit-Attention: Ritik Raj <[email protected]>
