You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Sergey Shelukhin <se...@hortonworks.com> on 2018/03/20 02:48:02 UTC

Review Request 66167: HIVE-18952 Tez session disconnect and reconnect on HS2 HA failover

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

Review request for hive.


Repository: hive-git


Description
-------

see jira


Diffs
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java f8e715d96efd95a68239728aeb8b8cc79450c20c 
  llap-client/src/java/org/apache/hadoop/hive/registry/ServiceInstanceStateChangeListener.java cc1ba337523e200701f4ad108cf770988548c34f 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmInstance.java a8629479ec8b0044595c481dfe4d9fea1f26550e 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmRegistryImpl.java 3ff732d9b796190e515111391223922dcff07bbd 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java 7ca3548561d5d3f0289d04c2722c4e9bcd5f0a7c 
  llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java 82179645da00c5673bee97e7a8ae8dd917736520 
  pom.xml 5be30f669666e85225af32011abb33ddb7c17d53 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/SessionExpirationTracker.java df14f1122651738a884c64118d140ec44aca7497 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java 89954cba6709bc4c39af3dd7ade85bf8456aaa0e 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java a051f9019572285a8ba62e6b4f9ecead860ac22c 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolSession.java d3748edb8674f9fdf49563238a3c3524f8acefe1 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java 046ea19c7227c16ec9c71cf470652385347cc78b 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java f0e620c684519f73c08bfa49d45d244085cc8b65 
  ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java 20a5947291f71c22d5da6ea2740426e50366344b 
  service/src/java/org/apache/hive/service/server/HiveServer2.java bb92c443e2364d9925bad60b0b9df9db8fd9134d 


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


Testing
-------


Thanks,

Sergey Shelukhin


Re: Review Request 66167: HIVE-18952 Tez session disconnect and reconnect on HS2 HA failover

Posted by Eric Wohlstadter <ew...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66167/#review199620
-----------------------------------------------------------




pom.xml
Line 198 (original), 198 (patched)
<https://reviews.apache.org/r/66167/#comment279953>

    Did you intend to commit this change?


- Eric Wohlstadter


On March 21, 2018, 2:12 a.m., Sergey Shelukhin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66167/
> -----------------------------------------------------------
> 
> (Updated March 21, 2018, 2:12 a.m.)
> 
> 
> Review request for hive.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java dd94d4db87 
>   llap-client/src/java/org/apache/hadoop/hive/registry/ServiceInstanceStateChangeListener.java cc1ba33752 
>   llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmInstance.java a8629479ec 
>   llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmRegistryImpl.java 3ff732d9b7 
>   llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java 7ca3548561 
>   llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java 82179645da 
>   pom.xml 5be30f6696 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/SessionExpirationTracker.java df14f11226 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java 89954cba67 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java a051f90195 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolSession.java d3748edb86 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java 046ea19c72 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java f0e620c684 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java 20a5947291 
>   service/src/java/org/apache/hive/service/server/HiveServer2.java bb92c443e2 
>   standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java 9f822564bd 
> 
> 
> Diff: https://reviews.apache.org/r/66167/diff/2/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Shelukhin
> 
>


Re: Review Request 66167: HIVE-18952 Tez session disconnect and reconnect on HS2 HA failover

Posted by Sergey Shelukhin <se...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66167/
-----------------------------------------------------------

(Updated March 28, 2018, 9:03 p.m.)


Review request for hive.


Repository: hive-git


Description
-------

see jira


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java a6866e75d3 
  llap-client/src/java/org/apache/hadoop/hive/registry/ServiceInstanceStateChangeListener.java cc1ba33752 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmInstance.java a8629479ec 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmRegistryImpl.java 3ff732d9b7 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java 7ca3548561 
  llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java 82179645da 
  pom.xml 194a7d0bb1 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/SessionExpirationTracker.java df14f11226 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java 89954cba67 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java a051f90195 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolSession.java d3748edb86 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java 046ea19c72 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java f0e620c684 
  ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java 20a5947291 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 6308c5cd4f 


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

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


Testing
-------


Thanks,

Sergey Shelukhin


Re: Review Request 66167: HIVE-18952 Tez session disconnect and reconnect on HS2 HA failover

Posted by Sergey Shelukhin <se...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66167/
-----------------------------------------------------------

(Updated March 21, 2018, 2:12 a.m.)


Review request for hive.


Repository: hive-git


Description
-------

see jira


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java dd94d4db87 
  llap-client/src/java/org/apache/hadoop/hive/registry/ServiceInstanceStateChangeListener.java cc1ba33752 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmInstance.java a8629479ec 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmRegistryImpl.java 3ff732d9b7 
  llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java 7ca3548561 
  llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java 82179645da 
  pom.xml 5be30f6696 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/SessionExpirationTracker.java df14f11226 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPool.java 89954cba67 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java a051f90195 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolSession.java d3748edb86 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java 046ea19c72 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java f0e620c684 
  ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java 20a5947291 
  service/src/java/org/apache/hive/service/server/HiveServer2.java bb92c443e2 
  standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java 9f822564bd 


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

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


Testing
-------


Thanks,

Sergey Shelukhin