> On April 17, 2014, 11:06 a.m., Dmytro Sen wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ServiceResourceProvider.java, > > line 1147 > > <https://reviews.apache.org/r/20439/diff/2/?file=561591#file561591line1147> > > > > too much spaces ))
Thanks, I have another patch soon and I'll address this. - Nate ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20439/#review40651 ----------------------------------------------------------- On April 17, 2014, 10:26 a.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20439/ > ----------------------------------------------------------- > > (Updated April 17, 2014, 10:26 a.m.) > > > Review request for Ambari, Dmytro Sen, Myroslav Papirkovskyy, and Sid Wagle. > > > Bugs: AMBARI-5488 > https://issues.apache.org/jira/browse/AMBARI-5488 > > > Repository: ambari > > > Description > ------- > > * Added special Flume ServiceState instance because the rules are different > there > * Raise an error on the flume.py script when Flume instances are not running > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ServiceResourceProvider.java > 1931121 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume.py > 9ac23d9 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume_handler.py > f796435 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ServiceResourceProviderTest.java > ee8e1ba > ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py 7a2ba8b > > Diff: https://reviews.apache.org/r/20439/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 14:07.713s > [INFO] Finished at: Wed Apr 16 20:40:37 EDT 2014 > [INFO] Final Memory: 20M/123M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Nate Cole > >
