eolivelli commented on PR #19727: URL: https://github.com/apache/pulsar/pull/19727#issuecomment-1465687715
@poorbarcode @BewareMyPower I support this patch, so +1 to it but this change is a "API change", as we are adding a new method to the [ManagedLedgerInterceptor] (https://github.com/apache/pulsar/pull/19727/files) API We should have discussed about this change on the mailing, a PIP is not needed, but we cannot do such changes without discussing them with the community. -- 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]
