----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41021/#review109099 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/checks/InstallPackagesCheck.java (line 64) <https://reviews.apache.org/r/41021/#comment168542> I don't understand the original comparison meaning, looks like mistake - Dmitro Lisnichenko On Dec. 6, 2015, 3:58 p.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41021/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2015, 3:58 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Jayush > Luniya, and Nate Cole. > > > Bugs: AMBARI-14235 > https://issues.apache.org/jira/browse/AMBARI-14235 > > > Repository: ambari > > > Description > ------- > > See > https://builds.apache.org/job/Ambari-branch-2.2/8/testReport/junit/org.apache.ambari.server.checks/InstallPackagesCheckTest/testPerform/ > > This failure seems consistent in the recent runs for branch-2.2. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/checks/InstallPackagesCheck.java > b80d6e6 > > ambari-server/src/test/java/org/apache/ambari/server/checks/InstallPackagesCheckTest.java > 18672e3 > > Diff: https://reviews.apache.org/r/41021/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Dmitro Lisnichenko > >
