Github user buggtb commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/382#discussion_r213355410
--- Diff:
bigtop-packages/src/charm/spark/layer-spark/lib/charms/layer/bigtop_spark.py ---
@@ -400,6 +400,10 @@ def start(self):
else
Github user buggtb commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/382#discussion_r213354825
--- Diff: bigtop-packages/src/charm/spark/layer-spark/reactive/spark.py ---
@@ -318,3 +318,10 @@ def client_present(client):
def client_should_stop
Github user buggtb commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/382#discussion_r213245996
--- Diff: bigtop-packages/src/charm/spark/layer-spark/reactive/spark.py ---
@@ -236,7 +235,8 @@ def reconfigure_spark():
# Almost all config changes
GitHub user buggtb opened a pull request:
https://github.com/apache/bigtop/pull/382
Bigtop-3070 support Spark Thrift Server in Charm
The thrift server has no exposure and should be configurable for Spark SQL
support etc.
You can merge this pull request into a Git repository