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

Review request for Ambari, Alexander Denissov, Alejandro Fernandez, bhuvnesh 
chaudhary, Goutam Tadi, Jaimin Jetly, jun aoki, Jayush Luniya, Matt, Newton 
Alex, Oleksandr Diachenko, Robert Levas, Sumit Mohanty, and Srimanth Gunturi.


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


Repository: ambari


Description
-------

ConfigHelper.updateConfigType takes in config_type (e.g. hdfs-site), map of 
updates (key value pairs) and a list of removals (properties).

The current implementation is skipping the updates to configuration if the Map 
of updates is empty, even if there are properties to be removed.

This is causing problem during disabling of kerberos for some components (like 
HAWQ) because some config parameters that are present in kerberos.json must be 
removed.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 
d311017 
  
ambari-server/src/test/java/org/apache/ambari/server/state/ConfigHelperTest.java
 9fe0fc3 

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


Testing
-------

Test case added.


Thanks,

Lav Jain

Reply via email to