Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefix thread names #50

Merged
merged 1 commit into from
Jun 23, 2023
Merged

Prefix thread names #50

merged 1 commit into from
Jun 23, 2023

Conversation

antoine-gallix
Copy link
Contributor

@antoine-gallix antoine-gallix commented Jun 20, 2023

Thread names appear in logs. The default naming scheme is something like ThreadPoolExecutor_x_y, with x enumerating ThreadPoolExecutor and y enumerating threads. This changes the thread name into something more recognizable, especially in environments where other named threads are operating. Worker_0, Worker_1, etc.

Faktory could also be an alternative naming. The standard seems to be camel case names for threads.

Thread names appear in logs. The default naming scheme is something like `ThreadPoolExecutor_x_y`, with `x` enumerating `ThreadPoolExecutor` and `y` enumerating threads. This changes the thread name into something more recognizable, especially in environments where other named threads are operating.
@cdrx cdrx merged commit e26e650 into cdrx:master Jun 23, 2023
@cdrx
Copy link
Owner

cdrx commented Jun 23, 2023

Thank you for your contributions @antoine-gallix

@cdrx cdrx mentioned this pull request Jun 23, 2023
@antoine-gallix antoine-gallix deleted the patch-4 branch June 26, 2023 17:41
@antoine-gallix
Copy link
Contributor Author

Great! Apart from doc improvements in other projects, I think it's my first code contribution ever to be merged! If you are curious to know where your code is running, we use your package in the NGO I work in to build this product. We have a ruby backend with a python data processing part, and we use faktory for background jobs.

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.

None yet

2 participants