ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitro Lisnichenko" <dlysniche...@hortonworks.com>
Subject Re: Review Request 29056: Repository Version Management > Create relation between repo_version and stack_version
Date Wed, 17 Dec 2014 13:30:44 GMT

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

Ship it!


looks good

- Dmitro Lisnichenko


On Dec. 17, 2014, 12:54 p.m., Yurii Shylov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29056/
> -----------------------------------------------------------
> 
> (Updated Dec. 17, 2014, 12:54 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate Cole.
> 
> 
> Bugs: AMBARI-8718
>     https://issues.apache.org/jira/browse/AMBARI-8718
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Instead of `stack` and `version` fields in `cluster_version` and `host_version` there
should be an FK pointing to repo_version.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/StacksService.java
4d0fb31 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
09f9b7f 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostStackVersionResourceProvider.java
e26a800 
>   ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/DistributeRepositoriesActionListener.java
428a0d2 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterVersionEntity.java
a7d4d24 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostVersionEntity.java
c090056 
>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
e3f4893 
>   ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java 8d2a902

>   ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java
2357004 
>   ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql b3cb6bd 
>   ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 720e253 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 08651eb 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql a91bc6a 
>   ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 968ebed 
>   ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
c6e5b26 
>   ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatMonitor.java
26bf0b9 
>   ambari-server/src/test/java/org/apache/ambari/server/api/services/StacksServiceTest.java
268c008 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
a7b2034 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackDefinedPropertyProviderTest.java
c31cc37 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
9b5a52a 
>   ambari-server/src/test/java/org/apache/ambari/server/events/EventsTest.java cc79f09

>   ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java fed9827

>   ambari-server/src/test/java/org/apache/ambari/server/orm/dao/ClusterVersionDAOTest.java
4b29635 
>   ambari-server/src/test/java/org/apache/ambari/server/orm/dao/HostVersionDAOTest.java
44bdf94 
>   ambari-server/src/test/java/org/apache/ambari/server/state/ServiceComponentTest.java
652299c 
>   ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java 6646b5e

>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
9cdb9ad 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersTest.java
0d55792 
>   ambari-server/src/test/java/org/apache/ambari/server/state/host/HostTest.java ab158e9

>   ambari-server/src/test/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostTest.java
2a04c71 
>   ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java
ee65c0c 
> 
> Diff: https://reviews.apache.org/r/29056/diff/
> 
> 
> Testing
> -------
> 
> [INFO] Ambari Groovy Shell ............................... SUCCESS [10.999s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 50:31.099s
> [INFO] Finished at: Mon Dec 15 15:04:55 CET 2014
> [INFO] Final Memory: 46M/674M
> 
> 
> Thanks,
> 
> Yurii Shylov
> 
>


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