Murtadha Hubail has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/3182

Change subject: [NO ISSUE][OTH] Allow Cancellation of Requests Before Job Start
......................................................................

[NO ISSUE][OTH] Allow Cancellation of Requests Before Job Start

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Currently clients' requests are marked cancellable only after
  their jobs start. This change allows cancellation of a request
  after its job  is compiled but before the job starts.

Change-Id: I8b7e811a16f37326d00d1b9d145819bdf6763fe0
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/82/3182/1

diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
index df285c6..80845f6 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
@@ -2603,11 +2603,11 @@
             if (jobSpec == null) {
                 return;
             }
-            final JobId jobId = JobUtils.runJob(hcc, jobSpec, jobFlags, false);
-            clientRequest.setJobId(jobId);
             if (cancellable) {
                 clientRequest.markCancellable();
             }
+            final JobId jobId = JobUtils.runJob(hcc, jobSpec, jobFlags, false);
+            clientRequest.setJobId(jobId);
             if (jId != null) {
                 jId.setValue(jobId);
             }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/3182
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8b7e811a16f37326d00d1b9d145819bdf6763fe0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <[email protected]>

Reply via email to