>From Ritik Raj <[email protected]>:
Attention is currently required from: Ali Alsuliman, Michael Blow.
Hello Ali Alsuliman, Jenkins, Michael Blow,
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 (#7).
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
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/dataflow/GlobalVirtualBufferCacheTest.java
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
5 files changed, 17 insertions(+), 7 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/67/20967/7
--
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: 7
Gerrit-Owner: Ritik Raj <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Ritik Raj <[email protected]>
Gerrit-CC: Anon. E. Moose #1000171
Gerrit-CC: Murtadha Hubail <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Michael Blow <[email protected]>
