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

Posted to commits@pulsar.apache.org by "rdhabalia (via GitHub)" <gi...@apache.org> on 2023/04/12 05:47:58 UTC

[GitHub] [pulsar] rdhabalia commented on issue #18950: PIP-229: Add a common interface to get fields of the MessageIdData

rdhabalia commented on issue #18950:
URL: https://github.com/apache/pulsar/issues/18950#issuecomment-1504691239

   as mentioned into PR: https://github.com/apache/pulsar/pull/19414
   we should not let users use `ledgerId/entryId. what if we replace bookkeeper with other some other storage? in that case, such codebase will be broken and exposing internal data and encouraging users to use it not a good design decision. I didn't review this PIP before but such discussion happened multiple times in past and Pulsar community had rejected this proposal in past. As we have not done any release yet, so, it's better to revert this PR soon.


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