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

Review request for Ambari, Alejandro Fernandez, Dmytro Grinenko, Jonathan 
Hurley, Nate Cole, Sumit Mohanty, and Sid Wagle.


Repository: ambari


Description
-------

Added support for "Appending" a value to a config through Upgrade, for tackling 
the following config change.


In hive-log4j content, the following needs to be added.

# Silence HadoopMetrics2Reporter INFO logs
log4j.logger.com.github.joshelser.dropwizard.metrics.hadoop=WARN,DRFA


This specific config change is valid for only HDP-2.5 . Hence the task has been 
added for HDP 2.3, 2.4 upgrade packs.

A similar problem is being discussed in https://reviews.apache.org/r/50508.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java
 f7de8a9 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ConfigUpgradeChangeDefinition.java
 54431eb 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ConfigureTask.java
 343c484 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/config-upgrade.xml 
c49e18e 
  
ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.5.xml
 133db26 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml 
ff5d4d9 
  ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/config-upgrade.xml 
473d8a0 
  
ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.5.xml
 d648638 
  ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml 
e67aebb 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ConfigureActionTest.java
 be0aeef 
  
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
 55cb23b 
  ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/config-upgrade.xml 
44f9e02 

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


Testing
-------

Added unit tests.

Manual Testing pending.


Thanks,

Aravindan Vijayan

Reply via email to