[
https://issues.apache.org/jira/browse/ROCKETMQ-172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15968825#comment-15968825
]
ASF GitHub Bot commented on ROCKETMQ-172:
-----------------------------------------
Github user Jaskey commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/90#discussion_r111552422
--- Diff:
client/src/main/java/org/apache/rocketmq/client/impl/MQClientAPIImpl.java ---
@@ -577,12 +577,12 @@ public void operationComplete(ResponseFuture
responseFuture) {
}
} else {
if (!responseFuture.isSendRequestOK()) {
- pullCallback.onException(new
MQClientException("send request failed", responseFuture.getCause()));
--- End diff --
But this is not a sl4j 's logging, it's a exception, so direct string is
needed.
> log improvement for rocketmq client
> -----------------------------------
>
> Key: ROCKETMQ-172
> URL: https://issues.apache.org/jira/browse/ROCKETMQ-172
> Project: Apache RocketMQ
> Issue Type: Improvement
> Components: rocketmq-client
> Reporter: Jaskey Lam
> Assignee: Jaskey Lam
>
> For some exception scenario, the log is not abundant enough to diagnose the
> problem.
> Some works should be done to improve the log messages
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
