Skip to content

Commit d748c1b

Browse files
author
zhousimin
committed
[Bugfix] Doris Sink should not throw exception when the transaction is already committed/visible
1 parent de9085d commit d748c1b

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

seatunnel-connectors-v2/connector-doris/src/main/java/org/apache/seatunnel/connectors/doris/sink/committer/DorisCommitter.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,8 @@ private void commitTransaction(DorisCommitInfo committable)
117117
String loadResult = EntityUtils.toString(response.getEntity());
118118
Map<String, String> res =
119119
mapper.readValue(loadResult, new TypeReference<HashMap<String, String>>() {});
120-
if (!LoadStatus.SUCCESS.equals(res.get("status"))) {
120+
if (!LoadStatus.SUCCESS.equals(res.get("status"))
121+
&& !ResponseUtil.isCommitted(res.get("msg"))) {
121122
log.error(
122123
"commit transaction error url:{},TxnId:{},result:{}",
123124
String.format(COMMIT_PATTERN, hostPort, committable.getDb()),

seatunnel-connectors-v2/connector-doris/src/main/java/org/apache/seatunnel/connectors/doris/util/ResponseUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,6 @@ public class ResponseUtil {
3131
+ "is already \\b(COMMITTED|committed|VISIBLE|visible)\\b, not pre-committed.");
3232

3333
public static boolean isCommitted(String msg) {
34-
return COMMITTED_PATTERN.matcher(msg).matches();
34+
return COMMITTED_PATTERN.matcher(msg).find();
3535
}
3636
}

0 commit comments

Comments
 (0)