>From Ali Alsuliman <[email protected]>:
Attention is currently required from: Ali Alsuliman, Hussain Towaileb, Michael
Blow, Murtadha Hubail, Murtadha Hubail.
Hello Anon. E. Moose #1000171, Hussain Towaileb, Jenkins, Michael Blow,
Murtadha Hubail, Murtadha Hubail,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21271?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Code-Review+2 by Murtadha Hubail, Integration-Tests+1 by Jenkins
Change subject: [ASTERIXDB-3649][API] Untrack non-ASYNC requests
......................................................................
[ASTERIXDB-3649][API] Untrack non-ASYNC requests
- user model changes: no
- storage format changes: no
- interface changes: yes
Details:
For a non-ASYNC request that has a statement not producing
a result, remove the tracking entry when the statement
execution is done.
For all other statements that failed, sweep the
result directory (remove job record entry + tracking entry)
- when a request is archived, nullify the thread executing
the request.
- on joblet clean-up in NCs, remove partial results if the
job has failed.
Ext-ref: MB-71997
Change-Id: I48f88ef268447f1bced0006087b21dc8bb1ff4c1
---
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/ClientRequest.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/IClientRequest.java
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/RequestTracker.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/result/ResultDirectoryService.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/Joblet.java
6 files changed, 87 insertions(+), 18 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/71/21271/3
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21271?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: I48f88ef268447f1bced0006087b21dc8bb1ff4c1
Gerrit-Change-Number: 21271
Gerrit-PatchSet: 3
Gerrit-Owner: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Attention: Murtadha Hubail <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Michael Blow <[email protected]>
Gerrit-Attention: Hussain Towaileb <[email protected]>
Gerrit-Attention: Murtadha Hubail
