[GitHub] [pulsar] liangyepianzhou commented on a change in pull request #14214: [Transaction] Call getCause() after exception wrapped by completeFuture

[GitBox <gi...@apache.org>]

liangyepianzhou commented on a change in pull request #14214:
URL: https://github.com/apache/pulsar/pull/14214#discussion_r803494617



##########
File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/transaction/TransactionCoordinatorClientImpl.java
##########
@@ -168,7 +168,7 @@ public TxnID newTransaction(long timeout, TimeUnit unit) throws TransactionCoord
         try {
             return newTransactionAsync(timeout, unit).get();
         } catch (Exception e) {
-            throw TransactionCoordinatorClientException.unwrap(e);
+            throw TransactionCoordinatorClientException.unwrap(e.getCause());

Review comment:
       Good idea! I will modify it immediately




-- 
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