You are viewing a plain text version of this content. The canonical link for it is here.

Posted to codereview@trafodion.apache.org by selvaganesang <gi...@git.apache.org> on 2018/04/09 21:40:20 UTC

[GitHub] trafodion pull request #1511: [TRAFODION-3009] Streamline error handling in ...

GitHub user selvaganesang opened a pull request:

    https://github.com/apache/trafodion/pull/1511

    [TRAFODION-3009] Streamline error handling in Executor utility commands

    get region stats command were populating the errors in the
    queue entry directly, but still used handleErrors() to populate the
    errors again in the HANDLE_ERROR_ state.  In some cases, the diagnostics
    area wasn't populated too.
    
    In CDH5.4, when the table is purged, the region info wasn't returned

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/selvaganesang/trafodion trafodion-3009_2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafodion/pull/1511.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1511
    
----
commit 4297cac937309138023d9dcc2612d6fcdc27c366
Author: selvaganesang <se...@...>
Date:   2018-04-09T21:07:22Z

    [TRAFODION-3009] Streamline error handling in Executor utility commands
    
    get region stats command were populating the errors in the
    queue entry directly, but still used handleErrors() to populate the
    errors again in the HANDLE_ERROR_ state.  In some cases, the diagnostics
    area wasn't populated too.
    
    In CDH5.4, when the table is purged, the region info wasn't returned

----


---

[GitHub] trafodion pull request #1511: [TRAFODION-3009] Streamline error handling in ...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/trafodion/pull/1511


---