You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tubemq.apache.org by GitBox <gi...@apache.org> on 2020/08/10 11:20:05 UTC

[GitHub] [incubator-tubemq] gosonzhang opened a new pull request #234: [TUBEMQ-311] Feedback more production information

gosonzhang opened a new pull request #234:
URL: https://github.com/apache/incubator-tubemq/pull/234


   


----------------------------------------------------------------
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] [incubator-tubemq] gosonzhang commented on pull request #234: [TUBEMQ-311] Feedback more production information

Posted by GitBox <gi...@apache.org>.
gosonzhang commented on pull request #234:
URL: https://github.com/apache/incubator-tubemq/pull/234#issuecomment-671301172


   The main changes are as follows:
   1. Add 3 parameters in SendMessageResponseB2P: messageId, appendTime, appendOffset, respectively save the messageId of the message storage, the time to save the message, and the index offset of the save message;
   2. Adjust the generation method of messageId, which is globally unique by the broker, and adjusted to be unique within the topic: messageId only needs to be unique within the specified topic, reducing the conflict processing of data generation;
   3. Modify MessageSentResult and add 2 parameters appendTime and appendOffset for the caller to use.


----------------------------------------------------------------
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] [incubator-tubemq] gosonzhang merged pull request #234: [TUBEMQ-311] Feedback more production information

Posted by GitBox <gi...@apache.org>.
gosonzhang merged pull request #234:
URL: https://github.com/apache/incubator-tubemq/pull/234


   


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