nahguam commented on PR #16715: URL: https://github.com/apache/pulsar/pull/16715#issuecomment-1197834940
> I'm unsure if adding the test coverage to actual usage is necessary. I mean to create the consumer with different priorities and then consume messages from the broker. Not just test the client configuration. I would hope there are already tests to verify the priority level behaviour. This only changes how the priority level is wired in from the configuration. I added a test to `ConsumerImplTest` to verify the wiring. -- 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]
