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}"

Reply via email to