Github user kwmonroe commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/382#discussion_r214485628
--- Diff: bigtop-packages/src/charm/spark/layer-spark/reactive/spark.py ---
@@ -318,3 +318,10 @@ def client_present(client):
def client_should_stop(client):
if is_state('leadership.is_leader'):
client.clear_spark_started()
+
+@when('spark.started', 'config.changed.spark_enable_thriftserver')
+def start_thrift():
--- End diff --
add those open/close port bits here---
