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



##########
File path: 
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java
##########
@@ -910,6 +910,20 @@ public TransactionConflictException(String msg) {
         }
     }
 
+    /**
+     * Thrown when transaction meta store handler not exists.
+     */
+    public static class MetaStoreHandlerNotExistsException extends 
PulsarClientException {

Review comment:
       This exception is caused by `txnID.getMostSigBits()` > the actual number 
of TCs.
   1. Change `MetaStoreHandlerNotExistsException` to 
`InvalidMetaStoreHandlerException` or `InvalidTxnIdException`
   2. Because in a normal production environment, the situation where 
`mostSiqBits` is greater than the actual number of TCs will not occur. Consider 
removing `MetaStoreHandlerNotExistsException`
   




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to