-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35217/
-----------------------------------------------------------

Review request for Ambari and Robert Nettleton.


Bugs: AMBARI-11790
    https://issues.apache.org/jira/browse/AMBARI-11790


Repository: ambari


Description
-------

If a property exists at multiple levels in a configuration such as in a 
blueprint and in the stack, only the most advanced occurrence of the property 
is removed, which in this case would be the blueprint configuration. Instead, 
all occurrences of the property should be removed from the config hierarchy 
such that a subsequent call to getPropertyValue() for the removed property 
would return null.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/topology/Configuration.java
 b7b9343 
  
ambari-server/src/test/java/org/apache/ambari/server/topology/ConfigurationTest.java
 1b9734e 

Diff: https://reviews.apache.org/r/35217/diff/


Testing
-------

Functional Testing:
Ensured that provided blueprint has configuration properly resolved when 
deploying

Unit Testing:
- Added new tests
- Currently running full unit test suite and will update with results prior to 
merging


Thanks,

John Speidel

Reply via email to