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]>

Reply via email to