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

    https://github.com/apache/incubator-hawq/pull/1308#discussion_r149592641
  
    --- Diff: src/backend/cdb/dispatcher.c ---
    @@ -1414,9 +1415,12 @@ dispatch_cleanup(DispatchData *data)
        if (dispatcher_is_state_error(data))
    --- End diff --
    
    May we check the status of 'proc_exit_inprogress' earlier and simpler? That 
is:
    ```
    if (proc_exit_inprogress)
        return;
    ```


---

Reply via email to