You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@inlong.apache.org by GitBox <gi...@apache.org> on 2022/08/01 03:02:57 UTC

[GitHub] [inlong] aloyszhang opened a new issue, #5303: [Improve] simplify flush check for appendMsg in FileMsgStore

aloyszhang opened a new issue, #5303:
URL: https://github.com/apache/inlong/issues/5303

   ### Description
   
   When appending msg, in `FileMsgStore`,  we may need to flush the message(and metadata) to the disk according to the check logic  https://github.com/apache/inlong/blob/master/inlong-tubemq/tubemq-server/src/main/java/org/apache/inlong/tubemq/server/broker/msgstore/disk/MsgFileStore.java
   
   But these checks are mixed which is hard to comprehend.
   
   We should simplify it in a concise manner.
   
   
   ### InLong Component
   
   InLong TubeMQ
   
   ### Are you willing to submit PR?
   
   - [X] Yes, I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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: dev-unsubscribe@inlong.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [inlong] dockerzhang closed issue #5303: [Improve] simplify flush check for appendMsg in FileMsgStore

Posted by GitBox <gi...@apache.org>.

dockerzhang closed issue #5303: [Improve] simplify flush check for appendMsg in FileMsgStore
URL: https://github.com/apache/inlong/issues/5303


-- 
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: dev-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org