----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43640/#review119476 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/stack/JmxQuery.java (lines 33 - 37) <https://reviews.apache.org/r/43640/#comment180843> A whole new class for one method? Could just change the methods from MasterHostResolver. You can even make the methods protected and extend it for mocking out test cases. - Nate Cole On Feb. 16, 2016, 7:43 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43640/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2016, 7:43 p.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan > Hurley, and Nate Cole. > > > Bugs: AMBARI-15064 > https://issues.apache.org/jira/browse/AMBARI-15064 > > > Repository: ambari > > > Description > ------- > > RU/EU tries to resolve the hostnames for NameNode, HBASE Master, and > ResourceManager by using MasterHostResolver.java, which queries configs and > then JMX. > > Ambari DB stores the hostnames in all lowercase, so if the configs or JMX > store the hostnames in a different case, then Ambari will throw an exception > that the host could not be found. > > To fix this, convert all hostnames to lowercase. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java > 795dfa7 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > 70440fc > ambari-server/src/main/java/org/apache/ambari/server/stack/JmxQuery.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java > 561350b > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java > eb5bf62 > > Diff: https://reviews.apache.org/r/43640/diff/ > > > Testing > ------- > > Ran unit tests in UpgradeHelperTest.java > Verified that was able to start Express Upgrade. > > Waiting for full set of unit tests to run. > > > Thanks, > > Alejandro Fernandez > >
