gaoran10 opened a new pull request #8347:
URL: https://github.com/apache/pulsar/pull/8347


   ### Motivation
   
   
![image](https://user-images.githubusercontent.com/15029908/96908159-d38d3f80-14ce-11eb-9e52-ee066434d960.png)
   
   Use the above approach instead of the sidecar approach.
   
   ### Modifications
   
   1. Produce transaction messages to the topic partition.
   2. The commit marker needs to record the related message-id list of its 
transaction.
   3. When the dispatcher read a transaction marker, get the messages of the 
transaction by message-id list in the marker and send them to the consumer.
   2. TransactionBuffer doesn't maintain any index data.
   
   ### Verifying this change
   
   This change added tests and can be verified as follows:
   
     - *Added unit tests for produce transaction messages and end transaction 
and read the messages*
   
       **org.apache.pulsar.broker.transaction.topictb.TransactionProduceTest**
       **org.apache.pulsar.broker.transaction.topictb.TransactionConsumeTest**
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (no)
     - The public API: (yes)
     - The schema: (no)
     - The default values of configurations: (no)
     - The wire protocol: (yes)
     - The rest endpoints: (no)
     - The admin cli options: (no)
     - Anything that affects deployment: (no)
   
   ### Documentation
   
     - Does this pull request introduce a new feature? (yes / no)
   


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

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


Reply via email to