----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62294/#review185336 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/java/org/apache/ambari/server/scheduler/ExecutionScheduleManager.java Line 665 (original), 669 (patched) <https://reviews.apache.org/r/62294/#comment261641> If ambariWebResource is a class level reference why make it a param? ambari-server/src/main/java/org/apache/ambari/server/scheduler/ExecutionScheduleManager.java Lines 803 (patched) <https://reviews.apache.org/r/62294/#comment261640> Why static? - Sid Wagle On Sept. 13, 2017, 8:25 p.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62294/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2017, 8:25 p.m.) > > > Review request for Ambari, Myroslav Papirkovskyy, Sid Wagle, and Yusaku Sako. > > > Bugs: AMBARI-21944 > https://issues.apache.org/jira/browse/AMBARI-21944 > > > Repository: ambari > > > Description > ------- > > Batch requests(Rolling Restart, HSI enable/disable, > decommission/recommission) not working when Ambari is configured with Knox > proxy > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/scheduler/ExecutionScheduleManager.java > f3b3c415ee > > ambari-server/src/test/java/org/apache/ambari/server/scheduler/ExecutionScheduleManagerTest.java > f901bda2b7 > ambari-web/app/controllers/main/host/details.js ad2ac98498 > ambari-web/app/controllers/main/service/item.js 224f83bdd9 > ambari-web/app/mixins/main/service/configs/component_actions_by_configs.js > 140e7de641 > ambari-web/app/utils/batch_scheduled_requests.js 6d1e721d47 > > ambari-web/test/mixins/main/service/configs/component_actions_by_configs_test.js > 9a73a9a3e1 > > > Diff: https://reviews.apache.org/r/62294/diff/1/ > > > Testing > ------- > > Tested manually on a cluster. > verified that ambari-web unit test and ExecutionScheduleManagerTest test > passes successfully: > > 30490 passing (20s) > 157 pending > > > Thanks, > > Jaimin Jetly > >
