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


Ship it!




Ship It!

- DIPAYAN BHOWMICK


On April 20, 2016, 7:47 a.m., Gaurav Nagar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46271/
> -----------------------------------------------------------
> 
> (Updated April 20, 2016, 7:47 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Nitiraj 
> Rathore, Pallav Kulshreshtha, Rohit Choudhary, and Ashwin Rajeev.
> 
> 
> Bugs: AMBARI-15919
>     https://issues.apache.org/jira/browse/AMBARI-15919
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Added ability to migrate data using rest API.
> Added data-migrator-class, data-version tags in view.xml . View needs to 
> implement ViewDataMigrator class that contains logic to migrate data from 
> older version.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/api/services/ViewDataMigrationService.java
>  PRE-CREATION 
>   
> ambari-server/src/main/java/org/apache/ambari/server/api/services/ViewInstanceService.java
>  1eea8db 
>   
> ambari-server/src/main/java/org/apache/ambari/server/orm/dao/PrivilegeDAO.java
>  772d538 
>   
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ViewInstanceEntity.java
>  2555f93 
>   
> ambari-server/src/main/java/org/apache/ambari/server/view/ViewDataMigrationContextImpl.java
>  PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java 
> 7379f37 
>   
> ambari-server/src/main/java/org/apache/ambari/server/view/configuration/ViewConfig.java
>  728f620 
>   
> ambari-server/src/test/java/org/apache/ambari/server/api/services/ViewDataMigrationServiceTest.java
>  PRE-CREATION 
>   
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/ViewInstanceEntityTest.java
>  c8c15da 
>   
> ambari-server/src/test/java/org/apache/ambari/server/view/ViewDataMigrationContextImplTest.java
>  PRE-CREATION 
>   
> ambari-server/src/test/java/org/apache/ambari/server/view/configuration/ViewConfigTest.java
>  14e5d3c 
>   ambari-views/examples/README.md 650f650 
>   ambari-views/examples/calculator-view/pom.xml d5e0e8e 
>   ambari-views/examples/cluster-view/pom.xml 99582a6 
>   ambari-views/examples/favorite-view/pom.xml ba196f5 
>   ambari-views/examples/hello-servlet-view/pom.xml ed6fc25 
>   ambari-views/examples/hello-spring-view/pom.xml 3475ef4 
>   ambari-views/examples/helloworld-view/pom.xml aa89778 
>   ambari-views/examples/phone-list-upgrade-view/docs/index.md PRE-CREATION 
>   ambari-views/examples/phone-list-upgrade-view/pom.xml PRE-CREATION 
>   
> ambari-views/examples/phone-list-upgrade-view/src/main/java/org/apache/ambari/view/phonelist/DataMigrator.java
>  PRE-CREATION 
>   
> ambari-views/examples/phone-list-upgrade-view/src/main/java/org/apache/ambari/view/phonelist/PhoneListServlet.java
>  PRE-CREATION 
>   
> ambari-views/examples/phone-list-upgrade-view/src/main/java/org/apache/ambari/view/phonelist/PhoneUser.java
>  PRE-CREATION 
>   
> ambari-views/examples/phone-list-upgrade-view/src/main/resources/WEB-INF/web.xml
>  PRE-CREATION 
>   ambari-views/examples/phone-list-upgrade-view/src/main/resources/view.xml 
> PRE-CREATION 
>   ambari-views/examples/phone-list-view/pom.xml ead9095 
>   ambari-views/examples/pom.xml de55ded 
>   ambari-views/examples/property-validator-view/pom.xml 045a277 
>   ambari-views/examples/property-view/pom.xml ad4630b 
>   ambari-views/examples/restricted-view/pom.xml f7fa3d6 
>   ambari-views/examples/simple-view/pom.xml 015f192 
>   ambari-views/examples/weather-view/pom.xml a018258 
>   
> ambari-views/src/main/java/org/apache/ambari/view/migration/EntityConverter.java
>  PRE-CREATION 
>   
> ambari-views/src/main/java/org/apache/ambari/view/migration/ViewDataMigrationContext.java
>  PRE-CREATION 
>   
> ambari-views/src/main/java/org/apache/ambari/view/migration/ViewDataMigrationException.java
>  PRE-CREATION 
>   
> ambari-views/src/main/java/org/apache/ambari/view/migration/ViewDataMigrator.java
>  PRE-CREATION 
>   ambari-views/src/main/java/org/apache/ambari/view/validation/Validator.java 
> ee029a8 
>   ambari-views/src/main/resources/view.xsd c3ad711 
>   pom.xml f540099 
> 
> Diff: https://reviews.apache.org/r/46271/diff/
> 
> 
> Testing
> -------
> 
> Manual testing.
> 
> 
> Thanks,
> 
> Gaurav Nagar
> 
>

Reply via email to