Skip to content

add recurring feature that allows running a job every N seconds#78

Merged
gresrun merged 2 commits into
gresrun:masterfrom
argvk:recurring-jobs
Jun 8, 2015
Merged

add recurring feature that allows running a job every N seconds#78
gresrun merged 2 commits into
gresrun:masterfrom
argvk:recurring-jobs

Conversation

@argvk
Copy link
Copy Markdown
Contributor

@argvk argvk commented Jun 4, 2015

  • a recurring job has a schedule in zset with the next timestamp as the
    score
  • the frequency of the job is stored in a hash
  • on pop the frequency and the score are summed up to form the new score
    and added back to the zset
  • a delete recurring removes it from the zset and hash
  • add delete and pop operations run on a redis transaction

related issue #77

* a recurring job has a schedule in zset with the next timestamp as the
score
* the frequency of the job is stored in a hash
* on pop the frequency and the score are summed up to form the new score
and added back to the zset
* a delete recurring removes it from the zset and hash
* add delete and pop operations run on a redis transaction
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.62%) to 83.97% when pulling c5cafaa on argvk:recurring-jobs into 1f387b9 on gresrun:master.

@argvk argvk mentioned this pull request Jun 4, 2015
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.62%) to 83.97% when pulling 55e96c5 on argvk:recurring-jobs into 1f387b9 on gresrun:master.

@argvk
Copy link
Copy Markdown
Contributor Author

argvk commented Jun 4, 2015

@gresrun I've fixed the failing test case

gresrun added a commit that referenced this pull request Jun 8, 2015
add recurring feature that allows running a job every N seconds
@gresrun gresrun merged commit 18018bd into gresrun:master Jun 8, 2015
@argvk argvk deleted the recurring-jobs branch June 8, 2015 17:13
@argvk
Copy link
Copy Markdown
Contributor Author

argvk commented Jun 8, 2015

thanks! 👍

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.

3 participants