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

Review request for Ambari, Dmytro Grinenko, Di Li, Dmitro Lisnichenko, Jonathan 
Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe.


Bugs: AMBARI-21461
    https://issues.apache.org/jira/browse/AMBARI-21461


Repository: ambari


Description
-------

Spark in IOP 4.2 BigInsights stack uses script name "thrift_server.py" instead 
of "spark_thrift_server.py" used by the common service Spark and the IOP 4.2.5 
BigInsights stack.
This seemed to have caused the following error during migration EU where Spark 
thrift server failed to restart.
"""
Caught an exception while executing custom service command: <class 
'ambari_agent.AgentException.AgentException'>: 'Script 
/var/lib/ambari-agent/cache/common-services/SPARK/1.2.1/package/scripts/thrift_server.py
 does not exist'; 'Script 
/var/lib/ambari-agent/cache/common-services/SPARK/1.2.1/package/scripts/thrift_server.py
 does not exist'
"""
We should be able to fix it by renaming "thrift_server.py" used in BI 4.2 stack 
to "spark_thrift_server.py" and updating metainfo.xml files accordingly.


Diffs
-----

  
ambari-server/src/main/resources/stacks/BigInsights/4.0/services/SPARK/metainfo.xml
 02abc62 
  
ambari-server/src/main/resources/stacks/BigInsights/4.0/services/SPARK/package/scripts/spark_thrift_server.py
 PRE-CREATION 
  
ambari-server/src/main/resources/stacks/BigInsights/4.0/services/SPARK/package/scripts/thrift_server.py
 39e15d3 
  
ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SPARK/metainfo.xml
 7b6bae1 
  
ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SPARK/package/scripts/spark_thrift_server.py
 PRE-CREATION 
  
ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SPARK/package/scripts/thrift_server.py
 d1a8b67 


Diff: https://reviews.apache.org/r/60814/diff/1/


Testing
-------

to be tested in integration tests


Thanks,

Di Li

Reply via email to