>From Ali Alsuliman <[email protected]>:
Attention is currently required from: Ali Alsuliman, Hussain Towaileb, Ian
Maxon, Michael Blow, Murtadha Hubail.
Hello Hussain Towaileb, Ian Maxon, Jenkins, Michael Blow, Murtadha Hubail,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21283?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by Ali Alsuliman, Code-Review+2 by Michael Blow,
Integration-Tests+1 by Jenkins, Verified+1 by Jenkins
The change is no longer submittable: Code-Review and Verified are unsatisfied
now.
Change subject: [ASTERIXDB-3649][HYR][API] Call job record finish before
callback
......................................................................
[ASTERIXDB-3649][HYR][API] Call job record finish before callback
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- ensure calling job record finish to record
job end time before calling the result callback since the
result callback will use the startTime, endTime, ... of
the job record.
- Make elapsedTime for result REST API be the time spent
delivering the result.
Ext-ref: MB-71997
Change-Id: I8f2a79d24f06258c6444a6cfaa531223e465c23e
---
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/AbstractQueryApiServlet.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryResultApiServlet.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryStatusApiServlet.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
M
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/result/ResultJobRecord.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/result/ResultDirectoryService.java
6 files changed, 21 insertions(+), 16 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/83/21283/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21283?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: I8f2a79d24f06258c6444a6cfaa531223e465c23e
Gerrit-Change-Number: 21283
Gerrit-PatchSet: 2
Gerrit-Owner: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Attention: Murtadha Hubail <[email protected]>
Gerrit-Attention: Ian Maxon <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Michael Blow <[email protected]>
Gerrit-Attention: Hussain Towaileb <[email protected]>
