[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #212: [LIVY-11] Enable HA support

[GitBox <gi...@apache.org>]

ajbozarth commented on a change in pull request #212: [LIVY-11] Enable HA support
URL: https://github.com/apache/incubator-livy/pull/212#discussion_r325412516
 
 

 ##########
 File path: conf/livy.conf.template
 ##########
 @@ -100,6 +100,16 @@
 # on user request and then livy server classpath automatically.
 # livy.repl.enable-hive-context =
 
+# High Availability mode of Livy. Possible values:
+# off: Default. Turn off High Availability. 
+# on: Livy uses Zookeeper as a state store to ensure a livy server is always available with the
+#     correct state.
+# Must set livy.server.ha.zookeeper-url to configure HA
+# livy.server.ha.mode = off
 
 Review comment:
   is `off`/`on` how HA configs usually work? I feel like `livy.server.ha.mode.enabled = true|false` would be more consistent with the current configs. This isn't an issue, just an observation

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services