道君

> Leaving aside the issue above. > > It looks like you want to add `role` to the consumer stats, that is a good idea to track the relationship between...

@nodece I think it doesn't matter, expose role name will not affect anything

This lead to too many changes, can we just update Cursor#config when the config changed?

> > This lead to too many changes, can we just update Cursor#config when the config changed? > > Sounds like a good idea, but we need to add a...

It looks PersistentTopics already exposed CompressType of the message, but `Message` API didn't expose the method of getting compressType. Maybe we need a PIP for the issue, or cast Message...

NonPersistentTopic has no ledgers to persist messages, so we don't need to care about it. About SystemTopic, I remember there is a PR to fix it

Got it, https://github.com/apache/pulsar/pull/22497

> > Got it, #22497 > > Thanks! But I see the #22497 did not merge. In our scenario, due to system topic triggered offload, many exceptions were printed in...

@chunxiaozheng https://github.com/apache/pulsar/pull/22497 merged, let's close this PR

@shibd Hi baodi, I've addressed your comment, thanks!