Skip to content

Set random manager ProcessWID if empty - #83

Open
heitorgo1 wants to merge 1 commit into
contribsys:mainfrom
heitorgo1:main
Open

Set random manager ProcessWID if empty#83
heitorgo1 wants to merge 1 commit into
contribsys:mainfrom
heitorgo1:main

Conversation

@heitorgo1

Copy link
Copy Markdown

Set the manager ProcessWID to the same random value as the Faktory client's ProcessWID when one is not provided.

@mperham

mperham commented May 26, 2026

Copy link
Copy Markdown
Contributor

I'm trying to understand why we didn't set this before. In Faktory's model, a process can either be a job producer (using the basic Faktory client to push jobs) or a consumer (a worker process fetching jobs to execute). I'm unclear when faktory.RandomProcessWID and mgr.ProcessWID should be different or if there are usecases which this change might break. Faktory is really stable at this point and I don't want to introduce a subtle bug for someone.

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