ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Review Request 33416: Merge Configs on Upgrade Across Stack Version
Date Tue, 21 Apr 2015 22:23:35 GMT

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

Review request for Ambari and Jonathan Hurley.


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


Repository: ambari


Description
-------

* Add config merging when upgrading across stack versions


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/checks/ConfigurationMergeCheck.java
bcce006 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
483e169 
  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RepositoryVersionDAO.java 2ed9846

  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 148e234 
  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 706e493

  ambari-server/src/test/java/org/apache/ambari/server/checks/ConfigurationMergeCheckTest.java
64eaa10 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
6b691e0 
  ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test.xml PRE-CREATION


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


Testing
-------

Manual + automated

Tests run: 2874, Failures: 0, Errors: 01, Skipped: 17

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 22:43.708s
[INFO] Finished at: Tue Apr 21 18:02:19 EDT 2015
[INFO] Final Memory: 39M/569M
[INFO] ------------------------------------------------------------------------


Thanks,

Nate Cole


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message