yanghua commented on a change in pull request #1003:
URL: https://github.com/apache/incubator-kyuubi/pull/1003#discussion_r701706241
##########
File path:
kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
##########
@@ -442,6 +444,19 @@ object KyuubiConf {
.transform(_.toLowerCase(Locale.ROOT))
.createWithDefault(SaslQOP.AUTH.toString)
+ val FRONTEND_REST_BIND_HOST: OptionalConfigEntry[String] =
buildConf("frontend.rest.bind.host")
+ .doc("Hostname or IP of the machine on which to run the REST frontend
service.")
+ .version("1.4.0")
+ .stringConf
+ .createOptional
+
+ val FRONTEND_REST_BIND_PORT: ConfigEntry[Int] =
buildConf("frontend.rest.bind.port")
+ .doc("Port of the machine on which to run the REST frontend service.")
+ .version("1.4.0")
+ .intConf
+ .checkValue(p => p == 0 || (p > 1024 && p < 65535), "Invalid Port number")
+ .createWithDefault(10009)
Review comment:
Should we choose another default port to distinguish from the thrift
binary bind port?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
