[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-12-03 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r353572944 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/exceptions/TransactionM

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-12-03 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r353572089 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionLog.java ##

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350019792 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionLog.java ##

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350041641 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350035812 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350040883 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350029397 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r349979380 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350025462 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r349981445 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350050273 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350020267 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionLog.java ##

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350023649 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350036026 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r349981383 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350021225 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350022645 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350042099 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-25 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r350029476 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-20 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r348377382 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-20 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r348391180 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/exceptions/TxnStoreStat

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-20 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r348376518 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346147104 ## File path: pulsar-transaction/common/pom.xml ## @@ -40,12 +40,22 @@ protobuf-java ${pr

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346158483 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346189781 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346162726 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TxnMeta.java #

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346166028 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346194349 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346145938 ## File path: pulsar-common/src/main/proto/PulsarApi.proto ## @@ -645,6 +645,35 @@ enum TxnAction { ABORT = 1;

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346162797 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TxnMeta.java #

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346159426 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionMetadataStor

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346179273 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346198793 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346183256 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346169767 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346174710 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346193402 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346200834 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346189342 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346167345 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346189206 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346163715 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TxnSubscription.java #

[GitHub] [pulsar] codelipenghui commented on a change in pull request #5570: Transaction log implemention

2019-11-14 Thread GitBox
codelipenghui commented on a change in pull request #5570: Transaction log implemention URL: https://github.com/apache/pulsar/pull/5570#discussion_r346183113 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTrans