You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "abdullah alamoudi (Code Review)" <do...@asterixdb.incubator.apache.org> on 2018/05/11 21:38:48 UTC

Change in asterixdb[master]: [NO ISSUE][RT] Make start and cancel job uninterruptible

Hello Anon. E. Moose #1000171, Jenkins,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#2).

Change subject: [NO ISSUE][RT] Make start and cancel job uninterruptible
......................................................................

[NO ISSUE][RT] Make start and cancel job uninterruptible

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

Details:
- Previously, a Hyracks data connection that starts
  a job might be interrupted before it receives the
  job id. This would leak resources since that job
  will run even though, no one is going to read its
  result.
- Similarly, job cancellation can be interrupted and
  so the job which was meant to be cancelled will
  continue running.
- To avoid this, a new thread is added to Hyracks
  Client Connection which takes care of starting
  and cancelling of jobs. The thread submitting these
  requests will be un-interruptible until those calls
  return.

Change-Id: I27b2aaae902b19829bd2df2ae04c5e704f5ca8e8
---
M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/indexing/IndexingScheduler.java
M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java
M hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/client/HyracksConnection.java
3 files changed, 149 insertions(+), 16 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/39/2639/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2639
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I27b2aaae902b19829bd2df2ae04c5e704f5ca8e8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <ba...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>