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

Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.


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


Repository: ambari


Description
-------

In UpgradeCatalog classes, we have a code which will add new properties during 
ambari upgrade. When we are adding them, we don't copy property attributes. We 
should fix that.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 
62c3b61 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalog.java
 edb1606 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog160.java
 2f7c84a 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog161.java
 528c6b1 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog170.java
 c54039f 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java
 2c9bf30 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java
 3c274b2 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog211.java
 616d4c4 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog160Test.java
 303fa3a 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog161Test.java
 a3b41b8 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog170Test.java
 93f7f8c 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java
 484c18d 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog210Test.java
 e5ef3c0 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalogTest.java
 319b9fe 

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


Testing
-------

mvn clean test


Thanks,

Vitalyi Brodetskyi

Reply via email to