----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65498/#review199136 -----------------------------------------------------------
llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java Line 330 (original), 330 (patched) <https://reviews.apache.org/r/65498/#comment279441> Fixed method signature. A path can have only one instance. llap-client/src/java/org/apache/hadoop/hive/registry/impl/ServiceInstanceBase.java Lines 67 (patched) <https://reviews.apache.org/r/65498/#comment279442> UUID is static. This is for the unit test. llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java Lines 84 (patched) <https://reviews.apache.org/r/65498/#comment279443> these are defaults. llap and tez AM uses the defaults but HS2 will override or ignore some prefixes.. need c'tor override for all prefixes. LLAP path: /llap-sasl/hiveuser/hostname/workers/worker-0000000 HS2 path: /hs2ActivePassiveHA-sasl/instances/instance-0000000000 llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java Lines 168 (patched) <https://reviews.apache.org/r/65498/#comment279444> This is required for ZK based service discovery. Clients connecting using ZK based service discovery to HS2 will have namespace in the URL (cannot have subpaths like /ns/user/workers). JDBC URL will have hs2ActivePassiveHA as namespace. For HS2 the ACL provider path will be /hs2ActivePassiveHA-sasl/instances/ There is no per user restriction (JDBC URL doesn't get users). llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java Line 406 (original), 459 (patched) <https://reviews.apache.org/r/65498/#comment279446> IntelliJ rule violation :) final is unnecessary for static methods. llap-common/src/java/org/apache/hadoop/hive/llap/metrics/MetricsUtils.java Lines 27 (patched) <https://reviews.apache.org/r/65498/#comment279447> llap-common is not dependent on llap-client. Dropped.. llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java Line 1365 (original), 1375 (patched) <https://reviews.apache.org/r/65498/#comment279448> ServiceRegistry interface had LlapServiceInstanceSet which is specific to LLAP. ServiceRegistry is updated to return generic ServiceInstanceSet. HiveServer2HAInstanceSet extends it to add isLeader() interface on top of ServiceRegistry. - Prasanth_J On March 13, 2018, 6:41 p.m., Prasanth_J wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65498/ > ----------------------------------------------------------- > > (Updated March 13, 2018, 6:41 p.m.) > > > Review request for hive, Sergey Shelukhin and Thejas Nair. > > > Bugs: HIVE-18281 > https://issues.apache.org/jira/browse/HIVE-18281 > > > Repository: hive-git > > > Description > ------- > > HIVE-18281: HiveServer2 HA for LLAP and Workload Manager > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java aedd1ec > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestActivePassiveHA.java > PRE-CREATION > > itests/util/src/main/java/org/apache/hive/jdbc/miniHS2/AbstractHiveService.java > 6cab8cd > itests/util/src/main/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java 8bbf8a4 > > llap-client/src/java/org/apache/hadoop/hive/llap/registry/ServiceRegistry.java > 5d7f813 > > llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapFixedRegistryImpl.java > c88198f > > llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapRegistryService.java > 80a6aba > > llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java > 8339230 > > llap-client/src/java/org/apache/hadoop/hive/llap/security/LlapTokenClient.java > 32d5caa > llap-client/src/java/org/apache/hadoop/hive/registry/RegistryUtilities.java > PRE-CREATION > llap-client/src/java/org/apache/hadoop/hive/registry/ServiceInstance.java > 908b3bb > > llap-client/src/java/org/apache/hadoop/hive/registry/ServiceInstanceSet.java > 34fba5c > > llap-client/src/java/org/apache/hadoop/hive/registry/impl/ServiceInstanceBase.java > db3d788 > > llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmInstance.java > 0724cf5 > > llap-client/src/java/org/apache/hadoop/hive/registry/impl/TezAmRegistryImpl.java > 417e571 > > llap-client/src/java/org/apache/hadoop/hive/registry/impl/ZkRegistryBase.java > 17269dd > llap-common/src/java/org/apache/hadoop/hive/llap/metrics/MetricsUtils.java > 9666517 > > llap-ext-client/src/java/org/apache/hadoop/hive/llap/LlapBaseInputFormat.java > 0120639 > > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/services/impl/LlapWebServices.java > 58bf8dc > > llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java > 66de3b8 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java > 46cfe56 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java b98fb58 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/Utils.java bc438bb > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LlapClusterStateForCompile.java > a8d729d > ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezSessionPool.java > d261623 > > service/src/java/org/apache/hive/service/server/HS2ActivePassiveHARegistry.java > PRE-CREATION > > service/src/java/org/apache/hive/service/server/HS2ActivePassiveHARegistryClient.java > PRE-CREATION > service/src/java/org/apache/hive/service/server/HiveServer2.java b7ece2b > > service/src/java/org/apache/hive/service/server/HiveServer2HAInstanceSet.java > PRE-CREATION > service/src/java/org/apache/hive/service/server/HiveServer2Instance.java > PRE-CREATION > service/src/java/org/apache/hive/service/servlet/HS2LeadershipStatus.java > PRE-CREATION > service/src/java/org/apache/hive/service/servlet/HS2Peers.java PRE-CREATION > > > Diff: https://reviews.apache.org/r/65498/diff/3/ > > > Testing > ------- > > > Thanks, > > Prasanth_J > >