Skip to content

Use multiple temporary files for storing state#9

Open
coledot wants to merge 9 commits intomarkryall:masterfrom
coledot:use_tempfile_for_state
Open

Use multiple temporary files for storing state#9
coledot wants to merge 9 commits intomarkryall:masterfrom
coledot:use_tempfile_for_state

Conversation

@coledot
Copy link

@coledot coledot commented Oct 11, 2014

If one sets up multiple threads to watch projects on the same server, then each thread will share a state file. That's no good, so instead setup a separate state file for each instance. Also, throw them into a temporary dir for good measure.

Copy link
Author

Choose a reason for hiding this comment

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

unrelated; fixes specs

@markryall
Copy link
Owner

hey - i'm so sorry i didn't respond earlier - i turned off all github email notifications because of the PR noise from my workplace :(

Wouldn't this break the crontab based execution in that the state would be persisted to a new location on each execution (unless you had explicitly specified the location of the state file)?

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