[
https://issues.apache.org/jira/browse/TAJO-2048?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15111787#comment-15111787
]
ASF GitHub Bot commented on TAJO-2048:
--------------------------------------
Github user jinossy commented on a diff in the pull request:
https://github.com/apache/tajo/pull/936#discussion_r50494584
--- Diff: tajo-core/src/main/java/org/apache/tajo/querymaster/Stage.java ---
@@ -1415,8 +1434,9 @@ public StageState transition(Stage stage, StageEvent
stageEvent) {
stage.abort(StageState.KILLED);
return StageState.KILLED;
} else {
- LOG.error("Invalid State " + stage.getSynchronizedState() + "
State");
- stage.abort(StageState.ERROR);
+ TajoInternalError error = new TajoInternalError("Invalid State
" + stage.getSynchronizedState() + " State");
--- End diff --
@jihoonson
this else clause seems to never be executed. I will remove this code block
for clarify
> QueryMaster and TajoWorker should support the exception propagation
> -------------------------------------------------------------------
>
> Key: TAJO-2048
> URL: https://issues.apache.org/jira/browse/TAJO-2048
> Project: Tajo
> Issue Type: Improvement
> Components: QueryMaster, Worker
> Affects Versions: 0.11.0
> Reporter: Jinho Kim
> Assignee: Jinho Kim
> Fix For: 0.12.0, 0.11.1
>
>
> Error propagation system was refactored by TAJO-1625. But worker and query
> master is missing in TAJO-1625.
> This issue improve error propagation system in TajoWorker and QueryMaster
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
