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

Surveys: Run sync_jotforms every half-hour #29714

Merged
merged 1 commit into from Jul 16, 2019

Conversation

islemaster
Copy link
Contributor

Run the sync_jotforms task every half-hour instead of every hour.

sync_jotforms looks up JotForm submissions that we did not have placeholders for and ingests them into our dashboard database. (The fill_jotform_placeholders job that runs every minute handles forms that we do have placeholders for.) This makes it a fallback case, but we still get bug reports from workshops when this happens and it takes up to an hour for survey responses to show up. This usually runs very quick, so we're making it run every half-hour to reduce the customer impact when it's needed.

Impetus for this change: Slack conversation

Run the sync_jotforms task every half-hour instead of every hour.

sync_jotforms looks up JotForm submissions that we did _not_ have
placeholders for and ingests them into our dashboard database. (The
fill_jotform_placeholders job that runs every minute handles forms that
we _do_ have placeholders for.)  This makes it a fallback case, but we
still get bug reports from workshops when this happens and it takes up
to an hour for survey responses to show up.  This usually runs very
quick, so we're making it run every half-hour to reduce the customer
impact when it's needed.

Impetus for this change: [Slack
conversation](https://codedotorg.slack.com/archives/C0T10H26N/p1563226711047300)
@islemaster islemaster merged commit c51e058 into staging Jul 16, 2019
@islemaster islemaster deleted the sync-jotforms-every-half-hour branch July 16, 2019 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants