[GitHub] drill issue #1145: DRILL-6187: Exception in RPC communication between DataCl...

2018-03-12 Thread parthchandra 
Github user parthchandra commented on the issue:

https://github.com/apache/drill/pull/1145
  
+1


---

[GitHub] drill issue #1145: DRILL-6187: Exception in RPC communication between DataCl...

2018-03-09 Thread sohami 
Github user sohami commented on the issue:

https://github.com/apache/drill/pull/1145
  
Made all the changes and squashed changes into 2 commits


---

[GitHub] drill issue #1145: DRILL-6187: Exception in RPC communication between DataCl...

2018-03-08 Thread sohami 
Github user sohami commented on the issue:

https://github.com/apache/drill/pull/1145
  
@vrozov - Thanks for the feedback. Update the PR with latest changes. 
Please help to review.


---

[GitHub] drill issue #1145: DRILL-6187: Exception in RPC communication between DataCl...

2018-03-01 Thread sohami 
Github user sohami commented on the issue:

https://github.com/apache/drill/pull/1145
  
@vrozov - Please help to review this PR.
It address the concurrency issue during authentication of control/data 
client to server side. Rather than adding the connection into connection holder 
right after TCP connection is available, the listener for connection success is 
called after successful authentication (if needed).


---