----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25691/#review53518 -----------------------------------------------------------
Ship it! Ship It! - Dmytro Sen On Сен. 16, 2014, 1:24 п.п., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25691/ > ----------------------------------------------------------- > > (Updated Сен. 16, 2014, 1:24 п.п.) > > > Review request for Ambari and Dmytro Sen. > > > Bugs: AMBARI-7330 > https://issues.apache.org/jira/browse/AMBARI-7330 > > > Repository: ambari > > > Description > ------- > > To support [HIVE-7935](https://issues.apache.org/jira/browse/HIVE-7935), we're > adding the following new config params to HiveServer2: > 1\. **hive.server2.support.dynamic.service.discovery** > default value: false > config type: boolean > description: Whether HiveServer2 supports dynamic service discovery for its > clients. To support this, each instance of HiveServer2 currently uses > ZooKeeper to register itself, when it is brought up. JDBC/ODBC clients should > use the ZooKeeper ensemble: hive.zookeeper.quorum in their connection string. > > 2\. **hive.server2.zookeeper.namespace** > default value: "hiveserver2" > config type: String > description: The parent node in ZooKeeper used by HiveServer2 when supporting > dynamic service discovery. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/configuration/hive-site.xml > 8206a6a > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/configuration/hive-site.xml > 9057fbe > > ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/HIVE/configuration/hive-site.xml > 8b4f065 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/HIVE/configuration/hive-site.xml > 9beb715 > ambari-web/app/data/HDP2/site_properties.js edcb322 > ambari-web/app/data/site_properties.js 02fa670 > > Diff: https://reviews.apache.org/r/25691/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
