AuroraTwinkle commented on PR #22861: URL: https://github.com/apache/pulsar/pull/22861#issuecomment-2160981969
> I added some further comments to simplify MessageListenerExecutor. The lifecycle is managed by the application that provides the instance and that's why there shouldn't be any details of the executor. Behind the `MessageListenerExecutor` there might be multiple different queues / thread pools. Passing the message instance will allow using the key or any other properties of the message to do decisions when it is needed, for example in an executor that uses a priority queue implementation. > > This change will also need a PIP. Please check for other PIPs on the mailing list for an example https://lists.apache.org/list.html?d...@pulsar.apache.org . You can find mailing list joining instructions at https://pulsar.apache.org/contact/ . The PIP template is at https://github.com/apache/pulsar/blob/master/pip/TEMPLATE.md . Before the PIP is accepted, we cannot merge implementation PRs. Okay, thanks for your prompt reply, i will try my best! -- 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-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
