You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/03/23 08:28:44 UTC

[GitHub] [pulsar] gaoran10 commented on a change in pull request #14816: [improve][tool] Improve transaction perf logs

gaoran10 commented on a change in pull request #14816:
URL: https://github.com/apache/pulsar/pull/14816#discussion_r832976432



##########
File path: pulsar-testclient/src/main/java/org/apache/pulsar/testclient/PerformanceProducer.java
##########
@@ -785,7 +787,9 @@ private static void runProducer(int producerId,
                                     });
                         } else {
                             transaction.abort().thenRun(() -> {
-                                log.info("Abort transaction {}", transaction.getTxnID().toString());
+                                if (log.isDebugEnabled()) {
+                                    log.debug("Abort transaction {}", transaction.getTxnID().toString());
+                                }

Review comment:
       Please check the `exceptionally` method, the error log should be `Abort transaction`.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org