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 2022/06/02 13:35:08 UTC

[GitHub] [pulsar] horizonzy opened a new pull request, #15901: Support local batch memory metadata store.

horizonzy opened a new pull request, #15901:
URL: https://github.com/apache/pulsar/pull/15901

   Fixes #15900
   
   ### Motivation
   Support local batch memory metadata store.
   
   Need to update docs? 
   
   - [ ] `doc-not-needed` 
   (Please explain why)
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy commented on pull request #15901: Support local batch memory metadata store.

Posted by GitBox <gi...@apache.org>.
horizonzy commented on PR #15901:
URL: https://github.com/apache/pulsar/pull/15901#issuecomment-1146635968

   Hanle #15931 firstly, after it merged. This pr will rebase 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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] github-actions[bot] commented on pull request #15901: Support local batch memory metadata store.

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15901:
URL: https://github.com/apache/pulsar/pull/15901#issuecomment-1144874532

   @horizonzy:Thanks for your contribution. For this PR, do we need to update docs?
   (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md#documentation), which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy commented on pull request #15901: Support local batch memory metadata store.

Posted by GitBox <gi...@apache.org>.
horizonzy commented on PR #15901:
URL: https://github.com/apache/pulsar/pull/15901#issuecomment-1145029370

   > @horizonzy Can you explain what is the advantage of batching in the in-memory implementation and why we need 2 different implementations for the in-memory metadata store?
   
   Some situation need transaction, such as I need transaction to support new feature. When I code unit test, the local memory metadata didn't support 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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] merlimat commented on pull request #15901: Support local batch memory metadata store.

Posted by GitBox <gi...@apache.org>.
merlimat commented on PR #15901:
URL: https://github.com/apache/pulsar/pull/15901#issuecomment-1145015942

   @horizonzy Can you explain what is the advantage for batching in the in-memory implementation and why do we need 2 different implementations for in-memory metadata store?r


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] github-actions[bot] commented on pull request #15901: Support local batch memory metadata store.

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15901:
URL: https://github.com/apache/pulsar/pull/15901#issuecomment-1144898594

   @horizonzy:Thanks for providing doc info!


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy closed pull request #15901: Support local batch memory metadata store.

Posted by GitBox <gi...@apache.org>.
horizonzy closed pull request #15901: Support local batch memory metadata store.
URL: https://github.com/apache/pulsar/pull/15901


-- 
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: commits-unsubscribe@pulsar.apache.org

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