ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Re: Review Request 33489: RU: desired stack must be cascaded during upgrade
Date Thu, 23 Apr 2015 20:57:55 GMT


> On April 23, 2015, 4:43 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java,
line 363
> > <https://reviews.apache.org/r/33489/diff/1/?file=940543#file940543line363>
> >
> >     This is a minor thing, but my preference is to have separate methods for error
checking and updating DB.

Noted.  Due to forces beyond my control, that will have to wait as we have to get 2.2 ->
2.3 working asap.


- Nate


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


On April 23, 2015, 3:56 p.m., Nate Cole wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33489/
> -----------------------------------------------------------
> 
> (Updated April 23, 2015, 3:56 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.
> 
> 
> Bugs: AMBARI-10005
>     https://issues.apache.org/jira/browse/AMBARI-10005
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> * Cascade the desired stack id to sub-objects: service, component, host_component
> * New configs should only be added on upgrade, not downgrade (separate jira)
> * Non-reportable components should be handled in Finalize
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java 596525b

>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
82b7307 
>   ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java
d97ad61 
>   ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java 64fe765 
>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
f62e3f3 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
5de2eac 
> 
> Diff: https://reviews.apache.org/r/33489/diff/
> 
> 
> Testing
> -------
> 
> Manual 2.2.x -> 2.2.y and 2.2 -> 2.3 + automated:
> 
> Tests run: 2881, Failures: 0, Errors: 0, Skipped: 17
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 24:57.558s
> [INFO] Finished at: Thu Apr 23 15:49:20 EDT 2015
> [INFO] Final Memory: 40M/714M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Nate Cole
> 
>


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