Re: Review Request 68034: HIVE-19568 Active/Passive HS2 HA: Disallow direct connection to passive HS2 instance

2018-07-26 Thread Sergey Shelukhin

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68034/
---

(Updated July 26, 2018, 7:03 p.m.)


Review request for hive and Prasanth_J.


Repository: hive-git


Description
---

see jira


Diffs (updated)
-

  itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestActivePassiveHA.java 
4055f13fd0 
  
itests/hive-unit/src/test/java/org/apache/hive/service/cli/session/TestQueryDisplay.java
 8b28e2dec0 
  itests/util/src/main/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java 
a78dd739b1 
  service/src/java/org/apache/hive/service/cli/CLIService.java dbfaf7154e 
  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 
e9649824f1 
  
service/src/java/org/apache/hive/service/cli/thrift/EmbeddedThriftBinaryCLIService.java
 8b61874f93 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 432a341bef 
  service/src/test/org/apache/hive/service/auth/TestPlainSaslHelper.java 
14e28323e0 
  
service/src/test/org/apache/hive/service/cli/TestCLIServiceConnectionLimits.java
 5ecea9a08b 
  service/src/test/org/apache/hive/service/cli/TestCLIServiceRestore.java 
6b69d4d0af 
  
service/src/test/org/apache/hive/service/cli/TestRetryingThriftCLIServiceClient.java
 7bae62d977 
  
service/src/test/org/apache/hive/service/cli/session/TestPluggableHiveSessionImpl.java
 90237c088f 
  
service/src/test/org/apache/hive/service/cli/session/TestSessionGlobalInitFile.java
 af7a72e70f 
  
service/src/test/org/apache/hive/service/cli/session/TestSessionManagerMetrics.java
 d954692e98 


Diff: https://reviews.apache.org/r/68034/diff/5/

Changes: https://reviews.apache.org/r/68034/diff/4-5/


Testing
---


Thanks,

Sergey Shelukhin



Re: Review Request 68034: HIVE-19568 Active/Passive HS2 HA: Disallow direct connection to passive HS2 instance

2018-07-26 Thread Sergey Shelukhin

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68034/
---

(Updated July 26, 2018, 7 p.m.)


Review request for hive and Prasanth_J.


Repository: hive-git


Description
---

see jira


Diffs (updated)
-

  itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestActivePassiveHA.java 
4055f13fd0 
  
itests/hive-unit/src/test/java/org/apache/hive/service/cli/session/TestQueryDisplay.java
 8b28e2dec0 
  itests/util/src/main/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java 
a78dd739b1 
  service/src/java/org/apache/hive/service/cli/CLIService.java dbfaf7154e 
  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 
e9649824f1 
  
service/src/java/org/apache/hive/service/cli/thrift/EmbeddedThriftBinaryCLIService.java
 8b61874f93 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 432a341bef 
  service/src/test/org/apache/hive/service/auth/TestPlainSaslHelper.java 
14e28323e0 
  
service/src/test/org/apache/hive/service/cli/TestCLIServiceConnectionLimits.java
 5ecea9a08b 
  service/src/test/org/apache/hive/service/cli/TestCLIServiceRestore.java 
6b69d4d0af 
  
service/src/test/org/apache/hive/service/cli/TestRetryingThriftCLIServiceClient.java
 7bae62d977 
  
service/src/test/org/apache/hive/service/cli/session/TestPluggableHiveSessionImpl.java
 90237c088f 
  
service/src/test/org/apache/hive/service/cli/session/TestSessionGlobalInitFile.java
 af7a72e70f 
  
service/src/test/org/apache/hive/service/cli/session/TestSessionManagerMetrics.java
 d954692e98 


Diff: https://reviews.apache.org/r/68034/diff/4/

Changes: https://reviews.apache.org/r/68034/diff/3-4/


Testing
---


Thanks,

Sergey Shelukhin



Re: Review Request 68034: HIVE-19568 Active/Passive HS2 HA: Disallow direct connection to passive HS2 instance

2018-07-25 Thread Sergey Shelukhin

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68034/
---

(Updated July 25, 2018, 11:55 p.m.)


Review request for hive and Prasanth_J.


Repository: hive-git


Description
---

see jira


Diffs (updated)
-

  itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestActivePassiveHA.java 
4055f13fd0 
  itests/util/src/main/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java 
a78dd739b1 
  service/src/java/org/apache/hive/service/cli/CLIService.java dbfaf7154e 
  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 
e9649824f1 
  
service/src/java/org/apache/hive/service/cli/thrift/EmbeddedThriftBinaryCLIService.java
 8b61874f93 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 432a341bef 
  service/src/test/org/apache/hive/service/auth/TestPlainSaslHelper.java 
14e28323e0 
  
service/src/test/org/apache/hive/service/cli/TestCLIServiceConnectionLimits.java
 5ecea9a08b 
  service/src/test/org/apache/hive/service/cli/TestCLIServiceRestore.java 
6b69d4d0af 
  
service/src/test/org/apache/hive/service/cli/TestRetryingThriftCLIServiceClient.java
 7bae62d977 
  
service/src/test/org/apache/hive/service/cli/session/TestPluggableHiveSessionImpl.java
 90237c088f 
  
service/src/test/org/apache/hive/service/cli/session/TestSessionGlobalInitFile.java
 af7a72e70f 
  
service/src/test/org/apache/hive/service/cli/session/TestSessionManagerMetrics.java
 d954692e98 


Diff: https://reviews.apache.org/r/68034/diff/3/

Changes: https://reviews.apache.org/r/68034/diff/2-3/


Testing
---


Thanks,

Sergey Shelukhin



Re: Review Request 68034: HIVE-19568 Active/Passive HS2 HA: Disallow direct connection to passive HS2 instance

2018-07-24 Thread Sergey Shelukhin

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68034/
---

(Updated July 25, 2018, 1:43 a.m.)


Review request for hive and Prasanth_J.


Repository: hive-git


Description
---

see jira


Diffs (updated)
-

  itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestActivePassiveHA.java 
4055f13fd0 
  itests/util/src/main/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java 
a78dd739b1 
  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 
e9649824f1 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 432a341bef 


Diff: https://reviews.apache.org/r/68034/diff/2/

Changes: https://reviews.apache.org/r/68034/diff/1-2/


Testing
---


Thanks,

Sergey Shelukhin



Review Request 68034: HIVE-19568 Active/Passive HS2 HA: Disallow direct connection to passive HS2 instance

2018-07-24 Thread Sergey Shelukhin

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68034/
---

Review request for hive and Prasanth_J.


Repository: hive-git


Description
---

see jira


Diffs
-

  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 
e9649824f1 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 432a341bef 


Diff: https://reviews.apache.org/r/68034/diff/1/


Testing
---


Thanks,

Sergey Shelukhin