Skip to content

Commit a567b87

Browse files
committed
[fix][dingo-sdk] Add rpc error log
1 parent 85dfbe7 commit a567b87

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

java/dingo-sdk/src/main/java/io/dingodb/sdk/service/caller/ServiceCaller.java

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,7 @@
44
import io.dingodb.sdk.common.DingoClientException.InvalidRouteTableException;
55
import io.dingodb.sdk.common.DingoClientException.RequestErrorException;
66
import io.dingodb.sdk.common.utils.ErrorCodeUtils;
7-
import io.dingodb.sdk.service.JsonMessageUtils;
8-
import io.dingodb.sdk.service.Caller;
9-
import io.dingodb.sdk.service.ChannelProvider;
10-
import io.dingodb.sdk.service.Service;
11-
import io.dingodb.sdk.service.ServiceCallCycles;
7+
import io.dingodb.sdk.service.*;
128
import io.dingodb.sdk.service.entity.Message.Request;
139
import io.dingodb.sdk.service.entity.Message.Response;
1410
import io.dingodb.sdk.service.entity.error.Errno;
@@ -99,6 +95,8 @@ public <REQ extends Request, RES extends Response> RES call(
9995
errMsgs.compute(
10096
channel.authority() + ">>" + error.getErrmsg(), (k, v) -> v == null ? 1 : v + 1
10197
);
98+
log.error(JsonMessageUtils.toJson(methodName, requestId, request, response, options) +
99+
getRegionId(channelProvider));
102100
boolean isPreWriteFailed = isPreWriteFailed(request);
103101
switch (handler.onErrStrategy(
104102
isPreWriteRequestFailed(errCode, isPreWriteFailed) ?
@@ -158,6 +156,13 @@ private static <REQ extends Request> boolean isPreWriteFailed(REQ request) {
158156
return false;
159157
}
160158

159+
private String getRegionId(ChannelProvider channelProvider) {
160+
if (channelProvider instanceof RegionChannelProvider) {
161+
return ",regionId:" + ((RegionChannelProvider) channelProvider).getRegionId();
162+
}
163+
return "";
164+
}
165+
161166
private void waitRetry() {
162167
LockSupport.parkNanos(TimeUnit.SECONDS.toNanos(1));
163168
}

0 commit comments

Comments
 (0)