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

Posted to dev@drill.apache.org by KulykRoman <gi...@git.apache.org> on 2017/09/29 17:57:22 UTC

[GitHub] drill pull request #967: DRILL-5564: Added finally block for stopWait() to a...

GitHub user KulykRoman opened a pull request:

    https://github.com/apache/drill/pull/967

    DRILL-5564: Added finally block for stopWait() to avoid all situation…

    …s where Drill able to miss stopWait() in case of exceptions (it can lead to assertions).

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

    $ git pull https://github.com/KulykRoman/drill DRILL-5564

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

    https://github.com/apache/drill/pull/967.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 #967
    
----
commit c0e567cefe7ad9bf9745ed41fb87abba60dbf1b9
Author: Roman Kulyk <ro...@gmail.com>
Date:   2017-09-29T17:26:39Z

    DRILL-5564: Added finally block for stopWait() to avoid all situations where Drill able to miss stopWait() in case of exceptions (it can lead to assertions).

----


---

[GitHub] drill pull request #967: DRILL-5564: Added finally block for stopWait() to a...

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

Github user asfgit closed the pull request at:

    https://github.com/apache/drill/pull/967


---

[GitHub] drill issue #967: DRILL-5564: Added finally block for stopWait() to avoid al...

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

Github user priteshm commented on the issue:

    https://github.com/apache/drill/pull/967
  
    @parthchandra can you review this?


---