Skip to content
This repository was archived by the owner on Jun 21, 2022. It is now read-only.

bug 866459: Tweak retries and max jobs per process#29

Merged
groovecoder merged 2 commits into
mdn:masterfrom
lmorchard:866459-tweak-retries-and-max-jobs
May 2, 2013
Merged

bug 866459: Tweak retries and max jobs per process#29
groovecoder merged 2 commits into
mdn:masterfrom
lmorchard:866459-tweak-retries-and-max-jobs

Conversation

@lmorchard

Copy link
Copy Markdown
Contributor

This is a band-aid fix. It makes the service retry a failed macro 10 times instead of 3, and it allows workers to process 8 jobs instead of 64 before exiting. It also gives me a handle on the server instance from the telnet REPL, so I maybe can do some live debugging.

My working theory is that we're running into some resource limits from cumulative runs. Lowering max jobs should help avoid that, and raising the retries should make failure less likely since the issue seems to be transient.

@lmorchard

Copy link
Copy Markdown
Contributor Author

Huh. Travis build failed, but I don't understand why. Seems like it didn't pick up the hirelings module

@lmorchard

Copy link
Copy Markdown
Contributor Author

Oh, I see what happened: I didn't have the latest version of that module working in kumascript

@groovecoder

Copy link
Copy Markdown
Contributor

Spot-check looks good. r+ merge away.

groovecoder added a commit that referenced this pull request May 2, 2013
bug 866459: Tweak retries and max jobs per process
@groovecoder groovecoder merged commit 0722dec into mdn:master May 2, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants