----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49128/#review139206 -----------------------------------------------------------
Ship it! Ship It! - Sumit Mohanty On June 23, 2016, 2:28 a.m., Swapan Shridhar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49128/ > ----------------------------------------------------------- > > (Updated June 23, 2016, 2:28 a.m.) > > > Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. > > > Bugs: AMBARI-17389 > https://issues.apache.org/jira/browse/AMBARI-17389 > > > Repository: ambari > > > Description > ------- > > Change done in the way we read : 'yarn.nodemanager.resource.memory-mb' and > 'yarn.scheduler.minimum-allocation-mb' for calculating LLAP configs. > > - Reads from: > - configurations (if changed as part of current Stack Advisor invocation > (output)), and services["changed-configurations"] > is empty -> Implies 1st invocation of SA, else > - services['configurations'] (input) -> Subsequent invocations from UI. > > - services["changed-configurations"] would be empty if Stack Advisor call is > made from Blueprints (1st invocation). Subsequent > Stack Advisor calls will have it non-empty. We do this because in > subsequent invocations, even if Stack Advsior calculates this > value (configurations), it is finally not recommended, making 'input' value > to survive. > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py > 8aa2f5f > ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py > fce1c1c > > Diff: https://reviews.apache.org/r/49128/diff/ > > > Testing > ------- > > Updated Python UTs. > > > Thanks, > > Swapan Shridhar > >