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

Posted to commits@pulsar.apache.org by "codelipenghui (via GitHub)" <gi...@apache.org> on 2023/10/07 03:14:06 UTC

Re: [PR] [fix][broker] Miss headersAndPayload and messageIdData in MessagePublishContext [pulsar]

codelipenghui commented on PR #21245:
URL: https://github.com/apache/pulsar/pull/21245#issuecomment-1751580685

   @nodece Only one person approved the PR, and the PR has no test covering the new changes. We should merge such PR. And the PR changed a public API which exposed to users. At least we should have a discussion under the mailing list.


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