Issue/PR created
PersistentTTLNode Thread leak · Issue #1263 · apache/curator
|
|
|
| | |
|
|
|
| |
PersistentTTLNode Thread leak · Issue #1263 · apache/curator
The recipe allows to be constructed using a provided ScheduledExecutorService
or using an internally created Exe...
|
|
|
En martes, 8 de abril de 2025, 14:07:22 CEST, Kezhu Wang
<[email protected]> escribió:
Hi,
Thank you for reporting this! I think you are right. Look forward to
the github issue and pr.
Best,
Kezhu Wang
On Tue, Apr 8, 2025 at 6:27 PM Evaristo José Camarero
<[email protected]> wrote:
>
> Hi there,
> I have been working on PersistentTTLNode recipe lately to fix
> https://github.com/apache/curator/issues/1258. I have realized that the
> recipe is leaking threads in a certain scenario.
> The recipe allows to be constructed using a provided ScheduledExecutorService
> or using an internally created Executor. The code tries to be generic and
> when closing the recipe is only cancelling recipe touch task, BUT the
> Executor is never shutdown. That means that when the Executor is internally
> created, the Executor Thread will remain there forever. In my view that is
> leak that should solved by distinguishing the case in which Executor is
> created internally or provided externally. In the first the case the Executor
> MUST be shutdown when the recipe is closed.
> Let me know if you agree with analysis and I will open a new ticket.
> Regards,
> /Cheva
