Skip to content

fix(cookie): make LocalCookie flush thread a named daemon - #2522

Merged
openai0229 merged 5 commits into
OtterMind:mainfrom
Aias00:fix/localcookie-daemon-thread-2518
Sep 11, 2026
Merged

openai0229 merged 5 commits into
OtterMind:mainfrom
Aias00:fix/localcookie-daemon-thread-2518

Conversation

@Aias00

@Aias00 Aias00 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What

Static initializer started a permanent while(true) cleanup loop as a non-daemon thread — JVM could not exit cleanly.

Location

chat2db-community-tools/.../http/LocalCookie.java:48-64

Fix

setDaemon(true) + name chat2db-local-cookie-flush (same pattern as #2386 ConnectionPool).

Fixes #2521

🤖 Generated with Claude Code

…2521)

The static initializer started a permanent while(true) cleanup loop as a
non-daemon thread, so the JVM could not exit cleanly once the class was
loaded. Mark it as a daemon and name it chat2db-local-cookie-flush.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: liuhy <liuhongyu@apache.org>
@Aias00
Aias00 requested a review from openai0229 as a code owner August 4, 2026 09:17
Copilot AI lite review requested due to automatic review settings August 4, 2026 09:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Aias00 added 2 commits August 19, 2026 05:27
Constraint: Preserve existing PR behavior while refreshing the merge base
Confidence: high
Scope-risk: narrow
@Aias00
Aias00 requested a review from a team as a code owner August 26, 2026 05:48
@openai0229
openai0229 merged commit bca9f30 into OtterMind:main Sep 11, 2026
15 of 16 checks passed
@openai0229 openai0229 moved this from In Review to Done in Chat2DB Community Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bug(cookie): LocalCookie cleanup thread is non-daemon, blocks JVM shutdown

3 participants