-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63695/
-----------------------------------------------------------
(Updated Nov. 10, 2017, 7:47 p.m.)
Review request for Ambari, Jonathan Hurley and Nate Cole.
Changes
-------
fixed review comments
Bugs: AMBARI-22387
https://issues.apache.org/jira/browse/AMBARI-22387
Repository: ambari
Description
-------
Ambari has removed its native support of distributing and installing LZO when
the LZO codecs are enabled in {{core-site}}. For existing clusters where LZO is
enabled, this means that performing an upgrade will now require manual user
intervention to get the LZO packages installed.
A pre-upgrade check should be created which checks to see if LZO is enabled in
the cluster and then produces a {{WARNING}} to the user letting them know that
before upgrading, they'd need to distribute the appropriate LZO packages.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java
f5e4ff9061
ambari-server/src/main/java/org/apache/ambari/server/checks/LZOCheck.java
PRE-CREATION
ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.6.xml
3d0d906856
ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.6.xml
3e044ec02e
ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.6.xml
c698d5e244
ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.6.xml
08725f04c4
ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.6.xml
48be5d1971
ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/upgrade-2.6.xml
cbe809a410
ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/nonrolling-upgrade-2.6.xml
b4e3745099
ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/upgrade-2.6.xml
98223904c8
ambari-server/src/test/java/org/apache/ambari/server/checks/LZOCheckTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/63695/diff/3/
Changes: https://reviews.apache.org/r/63695/diff/2-3/
Testing (updated)
-------
mvn clean test
live cluster check
Thanks,
Dmitro Lisnichenko
