You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by di...@apache.org on 2023/04/19 10:50:48 UTC
[doris-flink-connector] branch master updated: [fix] Fix the exception when formatting url (#131)
This is an automated email from the ASF dual-hosted git repository.
diwu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris-flink-connector.git
The following commit(s) were added to refs/heads/master by this push:
new 6e51865 [fix] Fix the exception when formatting url (#131)
6e51865 is described below
commit 6e51865be55b61b175e8e6870511e082763b3ef1
Author: gnehil <ad...@gmail.com>
AuthorDate: Wed Apr 19 18:50:42 2023 +0800
[fix] Fix the exception when formatting url (#131)
* [fix] Fix the exception caused by multi-fe node configuration when formatting url
* fix import *
---
.../src/main/java/org/apache/doris/flink/rest/RestService.java | 2 +-
.../doris/flink/sink/writer/JsonDebeziumSchemaSerializer.java | 6 ++++--
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/flink-doris-connector/src/main/java/org/apache/doris/flink/rest/RestService.java b/flink-doris-connector/src/main/java/org/apache/doris/flink/rest/RestService.java
index 7db5da3..b75586a 100644
--- a/flink-doris-connector/src/main/java/org/apache/doris/flink/rest/RestService.java
+++ b/flink-doris-connector/src/main/java/org/apache/doris/flink/rest/RestService.java
@@ -232,7 +232,7 @@ public class RestService implements Serializable {
* @throws IllegalArgumentException fe nodes is illegal
*/
@VisibleForTesting
- static String randomEndpoint(String feNodes, Logger logger) throws IllegalArgumentException {
+ public static String randomEndpoint(String feNodes, Logger logger) throws IllegalArgumentException {
logger.trace("Parse fenodes '{}'.", feNodes);
if (StringUtils.isEmpty(feNodes)) {
logger.error(ILLEGAL_ARGUMENT_MESSAGE, "fenodes", feNodes);
diff --git a/flink-doris-connector/src/main/java/org/apache/doris/flink/sink/writer/JsonDebeziumSchemaSerializer.java b/flink-doris-connector/src/main/java/org/apache/doris/flink/sink/writer/JsonDebeziumSchemaSerializer.java
index 60fca85..83b4f5e 100644
--- a/flink-doris-connector/src/main/java/org/apache/doris/flink/sink/writer/JsonDebeziumSchemaSerializer.java
+++ b/flink-doris-connector/src/main/java/org/apache/doris/flink/sink/writer/JsonDebeziumSchemaSerializer.java
@@ -25,6 +25,8 @@ import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.databind.node.JsonNodeFactory;
import org.apache.commons.codec.binary.Base64;
import org.apache.doris.flink.cfg.DorisOptions;
+import org.apache.doris.flink.exception.IllegalArgumentException;
+import org.apache.doris.flink.rest.RestService;
import org.apache.doris.flink.sink.HttpGetWithEntity;
import org.apache.flink.annotation.VisibleForTesting;
import org.apache.flink.util.StringUtils;
@@ -148,8 +150,8 @@ public class JsonDebeziumSchemaSerializer implements DorisRecordSerializer<Strin
}
}
- private boolean checkSchemaChange(String ddl) throws IOException {
- String requestUrl = String.format(CHECK_SCHEMA_CHANGE_API, dorisOptions.getFenodes(), database, table);
+ private boolean checkSchemaChange(String ddl) throws IOException, IllegalArgumentException {
+ String requestUrl = String.format(CHECK_SCHEMA_CHANGE_API, RestService.randomEndpoint(dorisOptions.getFenodes(), LOG), database, table);
Map<String,Object> param = buildRequestParam(ddl);
if(param.size() != 2){
return false;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org