You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Dmytro Sen <ds...@hortonworks.com> on 2014/08/18 16:41:31 UTC
Review Request 24801: Ambari upgrade: Accomodate renaming pig-content
to content in 1.7.0 ugradeCatalog
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24801/
-----------------------------------------------------------
Review request for Ambari and Andrew Onischuk.
Bugs: AMBARI-6891
https://issues.apache.org/jira/browse/AMBARI-6891
Repository: ambari
Description
-------
The pig.properties file is rendered in it's entirety with the name pig-content till 1.6.1 release. It has been renamed to content in 1.7.0 dev cycle.
This change in the property key name should be also handled in the cluster upgrade scenario
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog170.java fa47428
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog170Test.java 9325e9f
Diff: https://reviews.apache.org/r/24801/diff/
Testing
-------
Unit tests + manually checked that property name changed from pig-content to content
Thanks,
Dmytro Sen
Re: Review Request 24801: Ambari upgrade: Accomodate renaming
pig-content to content in 1.7.0 ugradeCatalog
Posted by Andrew Onischuk <ao...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24801/#review50880
-----------------------------------------------------------
Ship it!
Ship It!
- Andrew Onischuk
On Aug. 18, 2014, 2:41 p.m., Dmytro Sen wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24801/
> -----------------------------------------------------------
>
> (Updated Aug. 18, 2014, 2:41 p.m.)
>
>
> Review request for Ambari and Andrew Onischuk.
>
>
> Bugs: AMBARI-6891
> https://issues.apache.org/jira/browse/AMBARI-6891
>
>
> Repository: ambari
>
>
> Description
> -------
>
> The pig.properties file is rendered in it's entirety with the name pig-content till 1.6.1 release. It has been renamed to content in 1.7.0 dev cycle.
> This change in the property key name should be also handled in the cluster upgrade scenario
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog170.java fa47428
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog170Test.java 9325e9f
>
> Diff: https://reviews.apache.org/r/24801/diff/
>
>
> Testing
> -------
>
> Unit tests + manually checked that property name changed from pig-content to content
>
>
> Thanks,
>
> Dmytro Sen
>
>