This is an automated email from the ASF dual-hosted git repository. nic pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/kylin.git
The following commit(s) were added to refs/heads/master by this push: new 34a7172 KYLIN-4084 Reset kylin.stream.node in kylin-port-replace-util.sh 34a7172 is described below commit 34a717233784b8c34c6cf193e96ab8a5fe8b0856 Author: XiaoxiangYu <hit_la...@126.com> AuthorDate: Mon Jul 15 21:45:31 2019 +0800 KYLIN-4084 Reset kylin.stream.node in kylin-port-replace-util.sh In Kylin3.0, we introduce new real-time OLAP feature. When you use "kylin-port-replace-util.sh" to change listen port used by Kylin and use "kylin.sh start" to start a Coordinator. If you forget to set "kylin.stream.node" to new port, Coordinator will register its address into ZK using wrong port, that will make receiver cannot connect to Coordinator. That will create a mis-understand exception in all receivers and confused user. --- build/bin/kylin-port-replace-util.sh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/build/bin/kylin-port-replace-util.sh b/build/bin/kylin-port-replace-util.sh index 2c2ab40..45777da 100755 --- a/build/bin/kylin-port-replace-util.sh +++ b/build/bin/kylin-port-replace-util.sh @@ -89,6 +89,11 @@ then #replace ports in kylin.properties new_kylin_port=`expr ${KYLIN_DEFAULT_PORT} + ${OFFSET}` + #replace kylin.stream.node for Streaming Coordinator + stream_node="kylin.stream.node=`hostname -f`:$new_kylin_port" + echo "Using new kylin.stream.node: $stream_node" + sed -i "s/^kylin\.stream\.node=.*$/$stream_node/g" ${KYLIN_CONFIG_FILE} + sed -i "s/#*kylin.server.cluster-servers=\(.*\).*:\(.*\)/kylin.server.cluster-servers=\1:${new_kylin_port}/g" ${KYLIN_CONFIG_FILE} echo "New kylin port is : ${new_kylin_port}"