Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/2143
to look at the new patch set (#4).
Change subject: [ASTERIXDB-1871][ASTERIXDB-1871] Stop Consumer Thread on
Deallocate
......................................................................
[ASTERIXDB-1871][ASTERIXDB-1871] Stop Consumer Thread on Deallocate
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Currently there is a chance that a MaterializingPipelinedPartition
is deallocated before the consuming thread starts (e.g. due to job
abort) and therefore the consuming thread will not be interrupted
leading to leaked threads and files. This change checks if the
partition was deallocated before the consumer starts, then the
consumer thread cleans up any files then exits.
- Add test case.
Change-Id: I18c9fb085c149f41a202fff83aa6ec3aaeba6a77
---
M asterixdb/asterix-app/pom.xml
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
A
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/storage/DeallocatableTest.java
M
asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/management/NetworkingUtil.java
M
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/dataflow/TaskAttemptId.java
M
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/partitions/PartitionId.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/partitions/MaterializingPipelinedPartition.java
7 files changed, 118 insertions(+), 20 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/43/2143/4
--
To view, visit https://asterix-gerrit.ics.uci.edu/2143
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I18c9fb085c149f41a202fff83aa6ec3aaeba6a77
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
