drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sudheesh Katkam" <skat...@maprtech.com>
Subject Re: Review Request 34829: DRILL-3190: Invalid FragmentState transition from CANCELLATION_REQUESTED in QueryManager
Date Mon, 01 Jun 2015 17:54:44 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34829/
-----------------------------------------------------------

(Updated June 1, 2015, 5:54 p.m.)


Review request for drill, abdelhakim deneche, Chris Westin, and Jacques Nadeau.


Changes
-------

Addressed review comments
+ move fragment state transition logic to QueryManager


Bugs: DRILL-3190
    https://issues.apache.org/jira/browse/DRILL-3190


Repository: drill-git


Description (updated)
-------

DRILL-3190: Check for transitions from CANCELLATION_REQUESTED state
+ Moved state transition checks to QueryManager


Diffs (updated)
-----

  exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/FragmentData.java ceb77f0

  exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/QueryManager.java 71b77c6


Diff: https://reviews.apache.org/r/34829/diff/


Testing (updated)
-------

Successful unit and regression tests.


Thanks,

Sudheesh Katkam


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message