Overall I support this idea, but the name 'systemCursorsNames' is not good
and we should not make it configurable.

First, we have to make a call to the TTL thing, otherwise we will end up in
using that for other stuff, and not only TTL.

Second, we shouldn't make this configurable. Those cursors depend on the
version of Pulsar and users who upgrade from previous versions won't update
the configuration and this will lead to problems. And we cannot ask users
to update the configuration when updating

Enrico

Il Lun 6 Mag 2024, 08:02 Lari Hotari <lhot...@apache.org> ha scritto:

> LGTM, very useful. Thanks for driving this.
>
> -Lari
>
> On 2024/05/04 16:01:41 Hang Chen wrote:
> > Hi guys,
> >     I proposed a new proposal to improve the TTL impact on system
> > cursors. Please help take a look, thanks.
> >
> > PIP: https://github.com/apache/pulsar/pull/22651
> >
> > Thanks,
> > Hang
> >
>

Reply via email to