[ 
https://issues.apache.org/jira/browse/TAJO-1409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14496084#comment-14496084
 ] 

ASF GitHub Bot commented on TAJO-1409:
--------------------------------------

Github user jinossy commented on the pull request:

    https://github.com/apache/tajo/pull/430#issuecomment-93348821
  
    @jihoonson 
    done.run must be called once.  If throwing exception is allowed, I think 
that we need to handle all case error instead of InvocationFailure
    
    
https://issues.apache.org/jira/browse/TAJO-292?focusedCommentId=13838973&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13838973
    
    @navis 
    Thank you for your contribution. the patch is very useful but It seems to 
need failure test
    Could you add unit test ?


> Clients calling remote services returning BoolProto ignores false values
> ------------------------------------------------------------------------
>
>                 Key: TAJO-1409
>                 URL: https://issues.apache.org/jira/browse/TAJO-1409
>             Project: Tajo
>          Issue Type: Improvement
>          Components: rpc
>            Reporter: Navis
>            Assignee: Navis
>
> Some services like QueryMasterProtocolService returns false sometimes. But 
> clients are just ignoring them.



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

Reply via email to