Skip to content

feat: Include caller thread name in LimitedPoolIndexMerger threads.#19376

Merged
gianm merged 1 commit intoapache:masterfrom
gianm:segment-build-thread-names
Apr 25, 2026
Merged

feat: Include caller thread name in LimitedPoolIndexMerger threads.#19376
gianm merged 1 commit intoapache:masterfrom
gianm:segment-build-thread-names

Conversation

@gianm
Copy link
Copy Markdown
Contributor

@gianm gianm commented Apr 24, 2026

In Indexers (CliIndexer), a thread pool LimitedPoolIndexMerger is used to control the concurrency of IndexMerger operations and avoid excessive memory usage. This patch updates the thread names of this pool to reflect the calling thread, which will include the task ID.

In Indexers (CliIndexer), a thread pool LimitedPoolIndexMerger is used
to control the concurrency of IndexMerger operations and avoid
excessive memory usage. This patch updates the thread names of this
pool to reflect the calling thread, which will include the task ID.
Copy link
Copy Markdown
Member

@FrankChen021 FrankChen021 left a comment

Choose a reason for hiding this comment

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

The changes LGTM, no correctness issues found.

@gianm gianm merged commit 0ee04b4 into apache:master Apr 25, 2026
38 checks passed
@gianm gianm deleted the segment-build-thread-names branch April 25, 2026 23:26
@github-actions github-actions Bot added this to the 38.0.0 milestone Apr 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants