ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alejandro Fernandez" <afernan...@hortonworks.com>
Subject Re: Review Request 28759: Upgrade Execute: tie stage information more closely with UpgradeItem
Date Fri, 05 Dec 2014 20:04:40 GMT

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



ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java
<https://reviews.apache.org/r/28759/#comment106472>

    Some Javadoc would be nice.



ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
<https://reviews.apache.org/r/28759/#comment106419>

    If not needed, then line can be removed.



ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java
<https://reviews.apache.org/r/28759/#comment106475>

    How would this be changed?


- Alejandro Fernandez


On Dec. 5, 2014, 4:56 p.m., Nate Cole wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28759/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2014, 4:56 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Tom Beerbower.
> 
> 
> Bugs: AMBARI-8539
>     https://issues.apache.org/jira/browse/AMBARI-8539
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Ties Upgrade->Request and UpgradeItem->Stage more closely.  The properties of the
Request/Stage are pulled in, but assigned to the Upgrade/UpgradeItem response objects.  This
was preferred over having 2 "top level" objects (which didn't work with predicates anyway).
> 
> The method of pulling in the other objects is a bit unorthodox, and should be revisited
when we have time.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java
b5fe94e 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/UpgradeGroupService.java
da21658 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/UpgradeItemService.java
fb77853 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/UpgradeService.java
8b66491 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java
561f5d9 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StageResourceProvider.java
27ce015 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java
b7eb240 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeItemResourceProvider.java
c481dae 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
a2c30f7 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UpgradeDAO.java c08f809

>   ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 4320dbe

>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
fbb4f43 
> 
> Diff: https://reviews.apache.org/r/28759/diff/
> 
> 
> Testing
> -------
> 
> Tests run: 2323, Failures: 0, Errors: 0, Skipped: 18
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 18:05.831s
> [INFO] Finished at: Fri Dec 05 08:29:10 PST 2014
> [INFO] Final Memory: 30M/321M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Nate Cole
> 
>


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