Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Steffen911
Copy link
Member

@Steffen911 Steffen911 commented Sep 4, 2025

Important

Reorder freeAllTokenizers() call in onShutdown to occur after terminating tokenization worker threads in shutdown.ts.

  • Behavior:
    • Move freeAllTokenizers() call in onShutdown in shutdown.ts to occur after terminating tokenization worker threads.
    • Ensures tokenizers are freed from memory only after worker threads are terminated.
  • Logging:
    • Adjust log message order to reflect new shutdown sequence in shutdown.ts.

This description was created by Ellipsis for c08bbfc. You can customize this summary. It will automatically update as commits are pushed.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 4, 2025
@Steffen911 Steffen911 enabled auto-merge September 4, 2025 14:04
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Sep 4, 2025
@Steffen911 Steffen911 added this pull request to the merge queue Sep 4, 2025
@Steffen911 Steffen911 removed this pull request from the merge queue due to the queue being cleared Sep 4, 2025
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Sep 4, 2025
@Steffen911 Steffen911 added this pull request to the merge queue Sep 4, 2025
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Sep 4, 2025
Merged via the queue into main with commit 3cbe3bd Sep 4, 2025
30 checks passed
@Steffen911 Steffen911 deleted the steffen/terminate-tokenizer-before-free branch September 4, 2025 14:38
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant