drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Deneche A. Hakim (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (DRILL-3448) typo in QueryManager.DrillbitStatusListener will cause the Foreman to hang if a Drillbit dies
Date Thu, 02 Jul 2015 17:19:04 GMT

     [ https://issues.apache.org/jira/browse/DRILL-3448?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Deneche A. Hakim updated DRILL-3448:
------------------------------------
    Description: 
at the end of DrillbitStatusListener.drillbitUnregistered() there is if block:
{code}
      if (!atLeastOneFailure) {
        logger.warn("...");
        stateListener.moveToState(QueryState.FAILED,
            new ForemanException(...));
      }
{code}

this will basically fail the query if the drillbit DIDN'T contain any fragment for this Foreman,
which in fact should be the inverse.

  was:
at the end of DrillbitStatusListener.drillbitUnregistered() there is if block:
{code}
      if (!atLeastOneFailure) {
        logger.warn("...");
        stateListener.moveToState(QueryState.FAILED,
            new ForemanException(...));
      }
{code}

this will basically fail the query if the drillbit DIDN'T contain any fragment for this Foreman,
which in fact should be the inverse.

I will most likely fix this as part of DRILL-3167 as I need to make changes to the listener.


> typo in QueryManager.DrillbitStatusListener will cause the Foreman to hang if a Drillbit
dies
> ---------------------------------------------------------------------------------------------
>
>                 Key: DRILL-3448
>                 URL: https://issues.apache.org/jira/browse/DRILL-3448
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Execution - Flow
>    Affects Versions: 1.0.0
>            Reporter: Deneche A. Hakim
>            Assignee: Sudheesh Katkam
>            Priority: Critical
>             Fix For: 1.2.0
>
>
> at the end of DrillbitStatusListener.drillbitUnregistered() there is if block:
> {code}
>       if (!atLeastOneFailure) {
>         logger.warn("...");
>         stateListener.moveToState(QueryState.FAILED,
>             new ForemanException(...));
>       }
> {code}
> this will basically fail the query if the drillbit DIDN'T contain any fragment for this
Foreman, which in fact should be the inverse.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message