Jason918 commented on code in PR #18265:
URL: https://github.com/apache/pulsar/pull/18265#discussion_r1016108439
##########
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Producer.java:
##########
@@ -335,7 +348,7 @@ public TransportCnx getCnx() {
return this.cnx;
}
- private static final class MessagePublishContext implements
PublishContext, Runnable {
+ private static final class MessagePublishContext implements
PublishContext, Runnable, Position {
Review Comment:
It's tricky, `pulsar-broker` depends on `managed-ledger`, so we can't use
`MessagePublishContext` directly in `managed-ledger`, but I need to get
position info from this context.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
