This is an automated email from the ASF dual-hosted git repository.

zouxinyi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 1ff635cad8b [fix](arrow-flight-sql) Modify the default value of 
`arrow_flight_sql_port` configuration (#49236)
1ff635cad8b is described below

commit 1ff635cad8b65fd3529a0c2e92da6ea0d302f9f0
Author: Xinyi Zou <zouxi...@selectdb.com>
AuthorDate: Wed Mar 19 22:09:10 2025 +0800

    [fix](arrow-flight-sql) Modify the default value of `arrow_flight_sql_port` 
configuration (#49236)
    
    ### What problem does this PR solve?
    
    If your cluster 8081 and 8181 are occupied, FE and BE will not fail to
    start, but arrow flight server will be unavailable.
---
 be/src/common/config.cpp                                                | 2 +-
 conf/be.conf                                                            | 2 +-
 conf/fe.conf                                                            | 2 +-
 fe/fe-common/src/main/java/org/apache/doris/common/Config.java          | 2 +-
 .../src/test/java/org/apache/doris/utframe/MockedBackendFactory.java    | 2 +-
 fe/fe-core/src/test/java/org/apache/doris/utframe/MockedFrontend.java   | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/be/src/common/config.cpp b/be/src/common/config.cpp
index b349c7430b8..7552086dcde 100644
--- a/be/src/common/config.cpp
+++ b/be/src/common/config.cpp
@@ -62,7 +62,7 @@ DEFINE_Int32(be_port, "9060");
 // port for brpc
 DEFINE_Int32(brpc_port, "8060");
 
-DEFINE_Int32(arrow_flight_sql_port, "-1");
+DEFINE_Int32(arrow_flight_sql_port, "8050");
 
 // If the external client cannot directly access priority_networks, set 
public_host to be accessible
 // to external client.
diff --git a/conf/be.conf b/conf/be.conf
index bd01e4222f9..1d7573b4b7b 100644
--- a/conf/be.conf
+++ b/conf/be.conf
@@ -36,7 +36,7 @@ be_port = 9060
 webserver_port = 8040
 heartbeat_service_port = 9050
 brpc_port = 8060
-arrow_flight_sql_port = -1
+arrow_flight_sql_port = 8050
 
 # HTTPS configures
 enable_https = false
diff --git a/conf/fe.conf b/conf/fe.conf
index 5db098f289f..d2698d6f953 100644
--- a/conf/fe.conf
+++ b/conf/fe.conf
@@ -47,7 +47,7 @@ http_port = 8030
 rpc_port = 9020
 query_port = 9030
 edit_log_port = 9010
-arrow_flight_sql_port = -1
+arrow_flight_sql_port = 8070
 
 # Choose one if there are more than one ip except loopback address. 
 # Note that there should at most one ip match this list.
diff --git a/fe/fe-common/src/main/java/org/apache/doris/common/Config.java 
b/fe/fe-common/src/main/java/org/apache/doris/common/Config.java
index 509a56e334f..6fe73907f60 100644
--- a/fe/fe-common/src/main/java/org/apache/doris/common/Config.java
+++ b/fe/fe-common/src/main/java/org/apache/doris/common/Config.java
@@ -475,7 +475,7 @@ public class Config extends ConfigBase {
     public static int query_port = 9030;
 
     @ConfField(description = {"FE Arrow-Flight-SQL server 的端口号", "The port of 
FE Arrow-Flight-SQL server"})
-    public static int arrow_flight_sql_port = -1;
+    public static int arrow_flight_sql_port = 8070;
 
     @ConfField(description = {"MySQL 服务的 IO 线程数", "The number of IO threads in 
MySQL service"})
     public static int mysql_service_io_threads_num = 4;
diff --git 
a/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedBackendFactory.java 
b/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedBackendFactory.java
index 680b3e3641e..4446359b5f3 100644
--- 
a/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedBackendFactory.java
+++ 
b/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedBackendFactory.java
@@ -112,7 +112,7 @@ public class MockedBackendFactory {
     public static final int BE_DEFAULT_THRIFT_PORT = 9060;
     public static final int BE_DEFAULT_BRPC_PORT = 8060;
     public static final int BE_DEFAULT_HTTP_PORT = 8040;
-    public static final int BE_DEFAULT_ARROW_FLIGHT_SQL_PORT = 8070;
+    public static final int BE_DEFAULT_ARROW_FLIGHT_SQL_PORT = 8050;
 
     // create a mocked backend with customize parameters
     public static MockedBackend createBackend(String host, int heartbeatPort, 
int thriftPort, int brpcPort,
diff --git 
a/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedFrontend.java 
b/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedFrontend.java
index 6382621d905..651f3a0b5de 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedFrontend.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/utframe/MockedFrontend.java
@@ -79,7 +79,7 @@ public class MockedFrontend {
         MIN_FE_CONF.put("http_port", "8030");
         MIN_FE_CONF.put("rpc_port", "9020");
         MIN_FE_CONF.put("query_port", "9030");
-        MIN_FE_CONF.put("arrow_flight_sql_port", "9040");
+        MIN_FE_CONF.put("arrow_flight_sql_port", "8070");
         MIN_FE_CONF.put("edit_log_port", "9010");
         MIN_FE_CONF.put("priority_networks", "127.0.0.1/24");
         MIN_FE_CONF.put("sys_log_verbose_modules", "org");


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to