----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134511 -----------------------------------------------------------
ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py (line 221) <https://reviews.apache.org/r/47596/#comment199379> Can you explain how moving this line up helps ? Thx. - Swapan Shridhar On May 23, 2016, 8:20 a.m., Daniel Gergely wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47596/ > ----------------------------------------------------------- > > (Updated May 23, 2016, 8:20 a.m.) > > > Review request for Ambari, Laszlo Puskas, Oliver Szabo, Sandor Magyari, Sumit > Mohanty, and Sebastian Toader. > > > Bugs: AMBARI-16776 > https://issues.apache.org/jira/browse/AMBARI-16776 > > > Repository: ambari > > > Description > ------- > > Default value of hive.llap.daemon.yarn.container.mb is not aligned to > yarn.scheduler.minimum-allocation-mb > If the former one is less than the latter one, then hive server interactive > wont start. So the default values must be in a proper relation. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py > 9f5d799 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-site.xml > db5f616 > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py > 11aac72 > > Diff: https://reviews.apache.org/r/47596/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 2:15.746s > [INFO] Finished at: Thu May 19 18:52:43 CEST 2016 > [INFO] Final Memory: 58M/1120M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Daniel Gergely > >