----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31755/#review75323 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On March 4, 2015, 9:08 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31755/ > ----------------------------------------------------------- > > (Updated March 4, 2015, 9:08 p.m.) > > > Review request for Ambari, Jonathan Hurley and Nate Cole. > > > Bugs: AMBARI-9939 > https://issues.apache.org/jira/browse/AMBARI-9939 > > > Repository: ambari > > > Description > ------- > > Performing an RU will run Service Check groups after several major phases. > The Service Check group needs to include all services that can run a service > check. However, certain services like Pig still have service check scripts > even though they only have client components. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeContext.java > f8752a2 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ServiceCheckGrouping.java > e40706c > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java > 4a733b3 > ambari-server/src/test/resources/stacks/HDP/2.1.1/services/PIG/metainfo.xml > f310b70 > ambari-server/src/test/resources/stacks/HDP/2.1.1/services/TEZ/metainfo.xml > PRE-CREATION > > Diff: https://reviews.apache.org/r/31755/diff/ > > > Testing > ------- > > Performed an RU request and verified that the Service Check group included > one more service, namely Pig. > Waiting for unit test results. > > Local unit tests passed, > > OK > ---------------------------------------------------------------------- > Total run:609 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >
