[GitHub] drill issue #1103: DRILL-6124: Fixed possible NPE when no injection site is ...

2018-01-31 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/1103 You are right @arina-ielchiieva . Thanks for catching this, I will close the PR and mark the jira as invalid. ---

[GitHub] drill issue #1103: DRILL-6124: Fixed possible NPE when no injection site is ...

2018-01-31 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1103 @ilooner it looks like if latch is not found, execution control will return dummy latch [1]? If I am missing something, please explain. [1]

[GitHub] drill issue #1103: DRILL-6124: Fixed possible NPE when no injection site is ...

2018-01-30 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/1103 @sachouche @arina-ielchiieva ---