chore: AA-1058: Update edx-when version #29073
Conversation
|
jenkins run all |
Dillon-Dumesnil
force-pushed
the
ddumesnil/update-edx-when-version-aa-1058
branch
from
fa2f1f1
to
9558c35
Compare
Updates query counts for the additional query to look up the Schedule in edx-when
Dillon-Dumesnil
force-pushed
the
ddumesnil/update-edx-when-version-aa-1058
branch
from
9558c35
to
78fbd28
Compare
|
Your PR has finished running tests. There were no failures. |
| (ModuleStoreEnum.Type.mongo, 10, 164), | ||
| (ModuleStoreEnum.Type.split, 4, 160), | ||
| (ModuleStoreEnum.Type.mongo, 10, 227), | ||
| (ModuleStoreEnum.Type.split, 4, 210), |
There was a problem hiding this comment.
These and the test_resolvers change below are big bumps. That seems worrying? At first blush, this test doesn't appear to be doing a similar thing 50 times, but I didn't chase its logic fully.
There was a problem hiding this comment.
Dillon made https://openedx.atlassian.net/browse/AA-1081 to capture this issue as a follow-on item. In particular:
Although improving this is a great thing and we should do it/look into it, I do want to call out that the
_load_datesfunction uses the read replica so hopefully even the introduction of these queries doesn't have an effect on production.
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
|
EdX Release Notice: This PR has been deployed to the production environment. |
Related to openedx/edx-when#117