-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34478/
-----------------------------------------------------------
Review request for Ambari and Vitalyi Brodetskyi.
Bugs: AMBARI-11271
https://issues.apache.org/jira/browse/AMBARI-11271
Repository: ambari
Description
-------
Currently multiple customers are complaining on high services (re-)start time.
>From what we understood probably more than a minute is bad.
So after discussion with Mahadev it was decided to move out beeline check
which takes usually ~2 minutes (resulting in hive restarting in ~4minutes)
from start to service check, which seems a good place for that.
There is a problem due to Hive process asynchroniously starts and can drop
start without Ambari knowing it.
Also this was implemented some time ago because of:
AMBARI-5800
Race condition when starting all services causing Hive service check to fail
But, since now service check does exactly the beeline check with mutiple
retries this won't happen
Test scenarios included:
1\. Restart Hive and rt after it start service check
2\. Restart Hive and rt after it restart Webhcat
3\. Start service check with Hive Server HA
Diffs
-----
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs.py
418571a
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_datanode.py
d02bb39
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_namenode.py
453d824
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_nfsgateway.py
ac0e24d
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_snamenode.py
78ef977
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/journalnode.py
ada05e1
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py
3f1fb0d
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/utils.py
217f2f0
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml
32bc0ba
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_service.py
86312d2
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/service_check.py
04bd3b2
ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/kafka_broker.py
31d949a
ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox_gateway.py
956d09b
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service.py
3d6b8d1
ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper_server.py
d685d1d
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py
b58959a
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/shared_initialization.py
dd93818
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 4eb8178
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_journalnode.py 3fe2acb
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_namenode.py c753c39
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_nfsgateway.py 7255ea5
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_snamenode.py fd665c3
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_zkfc.py c2fdffa
ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py 90f3e06
ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py bc527e7
ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py 005b533
ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py
a2ee27f
ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py
ca8cf16
ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py
e752427
ambari-server/src/test/python/stacks/2.0.6/hooks/before-START/test_before_start.py
fb2699a
ambari-server/src/test/python/stacks/2.1/YARN/test_apptimelineserver.py
9e8b405
Diff: https://reviews.apache.org/r/34478/diff/
Testing
-------
mvn clean test
Thanks,
Andrew Onischuk
