Hello Luo Chen, Anon. E. Moose #1000171, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/2970
to look at the new patch set (#2).
Change subject: [NO ISSUE][STO] Ensure Files From Failed Bulkload Are Deleted
......................................................................
[NO ISSUE][STO] Ensure Files From Failed Bulkload Are Deleted
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- If a bulkload is interrupted at the end-stage (i.e. after all
tuples have been added but before the component is added to
the index's disk components), then the bulkload operation will
fail but the files from the failed operation are not deleted.
This change ensures that if a bulkload fails in such stage,
the files of the generated component are deleted to avoid
file is already mapped exception if the bulkload is attempted
again.
- Ensure disk component is only destroyed once in case of
bulkload cleanup.
- Stop attempting to write a checkpoint early if the failure was
due to the thread being interrupted.
Change-Id: I7f63054ac72a9482e779e49eb4da658a08fe7e9d
---
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/IndexCheckpointManager.java
M
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/ChainedLSMDiskComponentBulkLoader.java
M
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/LSMIndexDiskComponentBulkLoader.java
3 files changed, 34 insertions(+), 15 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/70/2970/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/2970
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7f63054ac72a9482e779e49eb4da658a08fe7e9d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: stabilization-f69489
Gerrit-Owner: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Luo Chen <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>