[GitHub] drill issue #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit may be ...

2017-10-19 Thread parthchandra 
Github user parthchandra commented on the issue:

https://github.com/apache/drill/pull/999
  
+1 to the second commit as well. This addresses the problem of the unit 
tests failing intermittently.


---

[GitHub] drill issue #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit may be ...

2017-10-18 Thread parthchandra 
Github user parthchandra commented on the issue:

https://github.com/apache/drill/pull/999
  
+1. 
You need a newline at the end :)


---

[GitHub] drill issue #999: DRILL-5881:Java Client: [Threat Modeling] Drillbit may be ...

2017-10-17 Thread sohami 
Github user sohami commented on the issue:

https://github.com/apache/drill/pull/999
  
@parthchandra - Please help to review this PR. I have added new unit tests 
for the change and made sure all the existing test's are also passing.


---