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/09/05 13:08:37 UTC

[GitHub] [pulsar] congbobo184 commented on a diff in pull request #17423: [improve][txn] Add getState in transaction for client API

congbobo184 commented on code in PR #17423:
URL: https://github.com/apache/pulsar/pull/17423#discussion_r962891048


##########
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/transaction/Transaction.java:
##########
@@ -29,6 +29,53 @@
 @InterfaceStability.Evolving
 public interface Transaction {
 
+    enum State {
+
+        /**
+         * When the transaction is in the `OPEN` state, it can produce with transaction and ack with the transaction.

Review Comment:
   yes, `messages can be produced and acked with this transaction` more accurate



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