-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63595/
-----------------------------------------------------------
Review request for Ambari, Dmitro Lisnichenko and Nate Cole.
Bugs: AMBARI-22370
https://issues.apache.org/jira/browse/AMBARI-22370
Repository: ambari
Description
-------
The wrapper scripts for services should be setting {{HADOOP_HOME}} correctly
regardless of whether or not an upgrade is taking place. There are many spots
which we current override this value and pass it in:
- Either via the Python files and setting the {{environment}}
- Or by setting it in configuration files
This should be removed since we must rely on correct {{HADOOP_HOME}} to be set
by the wrapper scripts.
Diffs
-----
ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py
d8f3d374f5
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
f6f678fa89
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
933515b01b
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_service_interactive.py
eaf95ad5db
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/webhcat_service.py
646090ac27
ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/package/scripts/service_check.py
a79b593ce8
ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py
400c87c828
ambari-server/src/main/resources/common-services/TEZ/0.4.0.2.1/package/scripts/params_linux.py
4d636855c3
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
12edc6924f
ambari-server/src/test/python/stacks/2.0.6/HIVE/test_webhcat_server.py
f9480eeaff
ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py
7f2ed464dc
ambari-server/src/test/python/stacks/2.1/TEZ/test_tez_client.py fad99f6496
ambari-server/src/test/python/stacks/2.3/MAHOUT/test_mahout_service_check.py
b80476cab2
ambari-server/src/test/python/stacks/2.5/HIVE/test_hive_server_int.py
4951c7e2ec
Diff: https://reviews.apache.org/r/63595/diff/1/
Testing
-------
OK
----------------------------------------------------------------------
Total run:1193
Total errors:0
Total failures:0
OK
Thanks,
Jonathan Hurley
