----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/ -----------------------------------------------------------
(Updated May 27, 2016, 4:39 p.m.) Review request for Ambari, Alejandro Fernandez, Daniel Gergely, and Sumit Mohanty. Changes ------- Updated review based on Daniel's comment. Bugs: AMBARI-16923 https://issues.apache.org/jira/browse/AMBARI-16923 Repository: ambari Description ------- - We need to read 'capacity-schdeuler' : 1. Firstly, from configurations where the configs can either be a dictionary or single "\n" string. It will be there if there is a change done to it as part of current invocation. If they are not there, then go ahead and read: 2. services for the configs where again it can either be a a dictionary or single "\n" string. - Removed stack_advisor.py 221-234 lines as they are no more valid as we dont do calulation based on change to cache value now. Diffs (updated) ----- ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 8c5351f ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py 0066e1d Diff: https://reviews.apache.org/r/47949/diff/ Testing ------- - Python UT added. - - Python UT passes. Thanks, Swapan Shridhar