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


Ship it!




Ship It!

- Andrew Onischuk


On May 17, 2016, 10:24 a.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47417/
> -----------------------------------------------------------
> 
> (Updated May 17, 2016, 10:24 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, and Sumit 
> Mohanty.
> 
> 
> Bugs: AMBARI-16683
>     https://issues.apache.org/jira/browse/AMBARI-16683
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> As i understood(according to discussion with Andrew O) we had some jira for 
> '*' to hdp version replacement in all packages. But here it was not replaced:
> {code}
> def get_lzo_packages(stack_version_unformatted):
>   lzo_packages = []
>  
>   if OSCheck.is_redhat_family() or OSCheck.is_suse_family():
>     lzo_packages += ["lzo", "hadoop-lzo-native"]
>   elif OSCheck.is_ubuntu_family():
>     lzo_packages += ["liblzo2-2"]
> 
>   if stack_version_unformatted and 
> check_stack_feature(StackFeature.ROLLING_UPGRADE, stack_version_unformatted):
>     lzo_packages += ["hadooplzo_*"]
>   else:
>     lzo_packages += ["hadoop-lzo"]
> 
>   return lzo_packages
> {code}
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/get_lzo_packages.py
>  e189d62 
>   
> ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/install_params.py
>  fe488c3 
>   
> ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py
>  784da9c 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
>  75924c5 
>   ambari-server/src/test/python/stacks/2.0.6/configs/client-upgrade.json 
> 47ab0a3 
>   ambari-server/src/test/python/stacks/2.0.6/configs/default.json f0e5208 
>   ambari-server/src/test/python/stacks/2.0.6/configs/nn_ru_lzo.json d44b002 
>   
> ambari-server/src/test/python/stacks/2.0.6/configs/ranger-namenode-start.json 
> b163a61 
>   
> ambari-server/src/test/python/stacks/2.0.6/hooks/after-INSTALL/test_after_install.py
>  6c7fe18 
>   ambari-server/src/test/python/stacks/2.1/HIVE/test_hive_metastore.py 
> c36b428 
>   
> ambari-server/src/test/python/stacks/2.2/configs/journalnode-upgrade-hdfs-secure.json
>  7b07d87 
>   ambari-server/src/test/python/stacks/2.2/configs/journalnode-upgrade.json 
> cf84bb7 
>   ambari-server/src/test/python/stacks/2.2/configs/oozie-downgrade.json 
> 7e5346c 
>   ambari-server/src/test/python/stacks/2.2/configs/oozie-upgrade.json 1c75d65 
> 
> Diff: https://reviews.apache.org/r/47417/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>

Reply via email to