Skip to content

Commit 98ced90

Browse files
authored
Merge pull request #537 from tapdata/TAP-6786
fix: TAP-6786 validate doris be nodes when testing connection write p…
2 parents be290d8 + e9397d1 commit 98ced90

File tree

1 file changed

+14
-6
lines changed
  • connectors/doris-connector/src/main/java/io/tapdata/connector/doris

1 file changed

+14
-6
lines changed

connectors/doris-connector/src/main/java/io/tapdata/connector/doris/DorisTest.java

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,18 +3,15 @@
33
import cn.hutool.http.HttpUtil;
44
import io.tapdata.common.CommonDbTest;
55
import io.tapdata.connector.doris.bean.DorisConfig;
6-
import io.tapdata.connector.doris.streamload.DorisStreamLoader;
76
import io.tapdata.kit.EmptyKit;
8-
import io.tapdata.kit.ErrorKit;
97
import io.tapdata.pdk.apis.entity.TestItem;
10-
import io.tapdata.pdk.apis.exception.testItem.TapTestConnectionEx;
11-
import io.tapdata.pdk.apis.exception.testItem.TapTestUnknownEx;
12-
import io.tapdata.pdk.apis.exception.testItem.TapTestVersionEx;
13-
import io.tapdata.pdk.apis.exception.testItem.TapTestWritePrivilegeEx;
8+
import io.tapdata.pdk.apis.exception.testItem.*;
9+
import io.tapdata.util.NetUtil;
1410
import org.apache.http.client.methods.HttpGet;
1511
import org.apache.http.impl.client.CloseableHttpClient;
1612
import org.apache.http.util.EntityUtils;
1713

14+
import java.io.IOException;
1815
import java.net.URI;
1916
import java.sql.Connection;
2017
import java.sql.SQLException;
@@ -71,6 +68,14 @@ protected Boolean testWritePrivilege() {
7168
jdbcContext.normalQuery("show backends", resultSet -> {
7269
while (resultSet.next()) {
7370
beCount.incrementAndGet();
71+
String beHost = (resultSet.getString("Host"));
72+
Integer httpPort = (resultSet.getInt("HttpPort"));
73+
if (null == beHost || null == httpPort) continue;
74+
try {
75+
NetUtil.validateHostPortWithSocket(beHost, httpPort);
76+
} catch (IOException e) {
77+
throw new TapTestHostPortEx(e, beHost, String.valueOf(httpPort));
78+
}
7479
}
7580
});
7681
} catch (SQLSyntaxErrorException e) {
@@ -82,6 +87,9 @@ protected Boolean testWritePrivilege() {
8287
beCount.set(1);
8388
}
8489
}
90+
} catch (TapTestHostPortEx e) {
91+
consumer.accept(new TestItem(TestItem.ITEM_WRITE, TestItem.RESULT_FAILED, "Validate BE nodes failed: " + e.getMessage()));
92+
return false;
8593
}
8694

8795
List<String> sqls = new ArrayList<>();

0 commit comments

Comments
 (0)