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 2020/12/11 15:33:59 UTC

[GitHub] [pulsar] reswqa opened a new pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

reswqa opened a new pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926


   fix #8925 
   
   
   ### Modifications
   
   Each partition sending a message must wait for the first message registerProducedTopic to complete.
   
   ### Verifying this change
   
   This change added tests and can be verified.
   
   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] sijie commented on pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926#issuecomment-755233689


   @gaoran10 @congbobo184 Can you review this?


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] reswqa commented on pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

Posted by GitBox <gi...@apache.org>.
reswqa commented on pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926#issuecomment-743264563


   @gaoran10 I'm not so sure this is the right modification, and there may be some problems beyond what I expected. Feel free to point out any wrong of my code. Thank you very much!


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] reswqa commented on pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

Posted by GitBox <gi...@apache.org>.
reswqa commented on pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926#issuecomment-755235748


   This issue should have been fixed in PR #9024 , I forgot to turn it off.
   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] reswqa closed pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

Posted by GitBox <gi...@apache.org>.
reswqa closed pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] reswqa edited a comment on pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

Posted by GitBox <gi...@apache.org>.
reswqa edited a comment on pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926#issuecomment-755235748


   This issue should have been fixed in PR #9024 , I forgot to close it.
   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [pulsar] reswqa commented on pull request #8926: [ISSUE 8925][transaction]Fixed the bug that messages may be out of order in transactions

Posted by GitBox <gi...@apache.org>.
reswqa commented on pull request #8926:
URL: https://github.com/apache/pulsar/pull/8926#issuecomment-743965454


   /pulsarbot run-failure-checks


----------------------------------------------------------------
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:
users@infra.apache.org