Skip to content

add instance id into thread name - #578

Merged
Edward Vaisman (eddyv) merged 4 commits into
confluentinc:masterfrom
flashmouse:therad_name
Jun 20, 2023
Merged

add instance id into thread name#578
Edward Vaisman (eddyv) merged 4 commits into
confluentinc:masterfrom
flashmouse:therad_name

Conversation

@flashmouse

@flashmouse flashmouse commented May 25, 2023

Copy link
Copy Markdown
Contributor

just add instance id into thread name, this would help debug

Checklist

  • Documentation (if applicable)
  • Changelog

@what-the-diff

what-the-diff Bot commented May 25, 2023 •
edited
Loading

Copy link
Copy Markdown

PR Summary

  • Improved thread naming in AbstractParallelEoSStreamProcessor

    • Set the thread name with an additional ID if available, to better identify and manage threads.

  • Enhanced thread naming in BrokerPollSystem

    • Set the thread name with an additional ID if available, for easier thread identification and management.

eddyv
Edward Vaisman (eddyv) approved these changes Jun 20, 2023
View reviewed changes

@eddyv Edward Vaisman (eddyv) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Thank you!

@eddyv
Edward Vaisman (eddyv) merged commit c333882 into confluentinc:master Jun 20, 2023
@astubbs Antony Stubbs (astubbs) mentioned this pull request Aug 5, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@eddyv Edward Vaisman (eddyv) eddyv approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@flashmouse @eddyv