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

ASF GitHub Bot commented on DRILL-4015:
---------------------------------------

Github user sudheeshkatkam commented on a diff in the pull request:

    https://github.com/apache/drill/pull/263#discussion_r45233348
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentStatusReporter.java
 ---
    @@ -98,7 +98,17 @@ void stateChanged(final FragmentState newState) {
       }
     
       private void sendStatus(final FragmentStatus status) {
    -    tunnel.sendFragmentStatus(status);
    +    FragmentStatus updatedStatus = status;
    +    // grab warnings from fragment context and pass them with fragment 
status
    +    if (context.getWarningHelper().hasWarnings()) {
    --- End diff --
    
    Can this be moved to the getStatus method?


> Update DrillClient and JDBC driver to expose warnings provided via RPC layer
> ----------------------------------------------------------------------------
>
>                 Key: DRILL-4015
>                 URL: https://issues.apache.org/jira/browse/DRILL-4015
>             Project: Apache Drill
>          Issue Type: Sub-task
>          Components: Client - JDBC, Execution - RPC
>            Reporter: Jacques Nadeau
>            Assignee: Abhijit Pol
>             Fix For: 1.4.0
>
>




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

Reply via email to