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

Fix #2063: convert continuous jobs into one-off jobs. #2320

Merged
merged 4 commits into from
Aug 3, 2016
Merged

Conversation

seanlip
Copy link
Member

@seanlip seanlip commented Aug 2, 2016

No description provided.

@seanlip seanlip changed the title Fix part of #2063: convert some continuous jobs into one-off jobs. Fix #2063: convert continuous jobs into one-off jobs. Aug 2, 2016
@codecov-io
Copy link

codecov-io commented Aug 2, 2016

Current coverage is 47.97% (diff: 100%)

Merging #2320 into develop will not change coverage

@@            develop      #2320   diff @@
==========================================
  Files           190        190          
  Lines         15953      15953          
  Methods        2740       2740          
  Messages          0          0          
  Branches       2702       2702          
==========================================
  Hits           7654       7654          
  Misses         8299       8299          
  Partials          0          0          

Powered by Codecov. Last update 30514fa...724fc8e

@wxyxinyu
Copy link
Contributor

wxyxinyu commented Aug 3, 2016

This looks great, thanks!

@wxyxinyu wxyxinyu merged commit 136e1ae into develop Aug 3, 2016
@wxyxinyu wxyxinyu deleted the one-off-jobs branch August 3, 2016 08:06
@@ -97,6 +99,27 @@ def get(self):
user_jobs_one_off.DashboardStatsOneOffJob.create_new())


class CronExplorationRecommendationsHandler(base.BaseHandler):
"""Handler for appending dashboard stats to a list."""
Copy link
Contributor

Choose a reason for hiding this comment

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

This comment seems to be incorrect.

Copy link
Member Author

Choose a reason for hiding this comment

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

Whoops, thanks for catching this, @526avijitgupta! Fixed in #2326, PTAL.

wxyxinyu pushed a commit that referenced this pull request Aug 8, 2016
* Change continuous recommendations job into a one-off cron job.

* Change continuous search ranker job to a one-off job.

* Fix lint issues.

* Decrease queue rate.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants