----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54552/#review158711 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java (line 296) <https://reviews.apache.org/r/54552/#comment229505> Maybe log some more context info before Runtime ex? ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java (line 315) <https://reviews.apache.org/r/54552/#comment229508> Does order of items in source and data matter here ? Please ignore if this comment doesn't apply. But if source is unique but 2 data items have same value. ['a', 'b', 'c'] -> ['d', 'd', 'f'] - Sid Wagle On Dec. 8, 2016, 10:59 p.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54552/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2016, 10:59 p.m.) > > > Review request for Ambari, Attila Magyar, Laszlo Puskas, Sumit Mohanty, and > Thejas Nair. > > > Bugs: AMBARI-19133 > https://issues.apache.org/jira/browse/AMBARI-19133 > > > Repository: ambari > > > Description > ------- > > `hadoop.proxyuser.HTTP.hosts` should not be updated when Hive is installed > unless WebHcat is installed. > > This is happening because the following block in the Kerberos descriptor is > at the HIVE service level rather than the WEBHCAT_SERVER component level. > > ``` > { > "core-site": { > "hadoop.proxyuser.HTTP.hosts": > "${clusterHostInfo/webhcat_server_host}" > } > }, > ``` > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java > d472b79 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json > 872bfac > ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/kerberos.json > 34bda73 > > ambari-server/src/test/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelperTest.java > 90f7022 > > Diff: https://reviews.apache.org/r/54552/diff/ > > > Testing > ------- > > Manually tested > > # Local test results: PENDING > > # Jenkins test results: PENDING > > > Thanks, > > Robert Levas > >
