Github user HanumathRao commented on the issue:
https://github.com/apache/drill/pull/1059
@amansinha100 These issues might not be because of this PR changes. I
think, these are random issues which are being shown up in my branch. I heard
from @Ben-Zvi that he also had some issues
Github user amansinha100 commented on the issue:
https://github.com/apache/drill/pull/1059
@HanumathRao can you rebase your PR on the latest master and run the
functional tests ? I saw the following test failures which seem related to
these changes:
Execution Failures:
Github user HanumathRao commented on the issue:
https://github.com/apache/drill/pull/1059
@vdiravka Thank you for the review comments.
I have done the needed changes.
---
Github user HanumathRao commented on the issue:
https://github.com/apache/drill/pull/1059
@paul-rogers Thanks for the review. I have moved the testcase to one test
file and called it from multiple places. I have also made changes to move the
code in checkForEarlyExit to base class.
