### _Why are the changes needed?_ This PR resolves https://github.com/apache/kyuubi/issues/5010, by introducing a new configuration property that makes it possible to override the hostname or ip of the Kyuubi server's thrift frontend service that will be registered for the service discovery ensemble. ### _How was this patch tested?_ - [X] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [X] Add screenshots for manual tests if appropriate Before applying the patch, when we set `kyuubi.frontend.bind.host=0.0.0.0`, the znode becomes as follows. ``` [zk: some_zk_host:some_zk_port(CONNECTED) 0] ls /some_kyuubi_zk_namespace [serviceUri=0.0.0.0:10009;version=1.8.0-SNAPSHOT;sequence=0000000007] ``` After applying the patch, when we set `kyuubi.frontend.bind.host=0.0.0.0` and `kyuubi.frontend.advertised.host=some_kyuubi_host`, it becomes as follows. ``` [zk: some_zk_host:some_zk_port(CONNECTED) 0] ls /some_kyuubi_zk_namespace [serviceUri=some_kyuubi_host:10009;version=1.8.0-SNAPSHOT;sequence=0000000009] ``` - [X] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request `./build/mvn clean install -pl kyuubi-common` succeeded. Closes #5015 from touchida/issue-5010-connection-url. Closes #5010 4598c2baf [Cheng Pan] Update kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala bf53a8a13 [Cheng Pan] Update kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala 0ddf09f0f [toshihiko.uchida] Use pattern matching instead of if-else 34c61d90d [toshihiko.uchida] Rename to kyuubi.frontend.advertised.host and apply to all frontends 7fd980baf [toshihiko.uchida] [KYUUBI #5010] Make Kyuubi server's thrift frontend connection URL configurable Lead-authored-by: toshihiko.uchida <toshihiko.uchida@linecorp.com> Co-authored-by: Cheng Pan <pan3793@gmail.com> Signed-off-by: Cheng Pan <chengpan@apache.org> |
||
|---|---|---|
| .. | ||
| spark | ||
| engine_lifecycle.md | ||
| engine_on_kubernetes.md | ||
| engine_on_yarn.md | ||
| engine_share_level.md | ||
| high_availability_guide.md | ||
| hive_metastore.md | ||
| index.rst | ||
| kyuubi_on_kubernetes.md | ||
| migration-guide.md | ||
| settings.md | ||