Skip to content

Conversation

@jansenk
Copy link
Contributor

@jansenk jansenk commented Jun 4, 2025

This change significantly speeds up this endpoint and fixes an issue a partner has been dealing with where a particularly large course causes a timeout.

some basic timing on a large local course:

pre-fix:
0:00:05.442233
0:00:04.578525
0:00:04.437510
0:00:04.769469
0:00:04.435430

post-fix:
0:00:00.958613
0:00:00.563015
0:00:00.540611
0:00:00.581324
0:00:00.560781

significant improvement

Copy link
Contributor

@nsprenkle nsprenkle left a comment

Choose a reason for hiding this comment

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

Nice!

@jansenk jansenk enabled auto-merge (squash) June 4, 2025 16:00
@jansenk jansenk merged commit f58c6f0 into master Jun 4, 2025
48 checks passed
@jansenk jansenk deleted the jkantor/bulkactions branch June 4, 2025 16:52
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

marlonkeating pushed a commit that referenced this pull request Jul 15, 2025
* fix: wrap sequence metadata view in bulk actions contextmanager

* style: line too long
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.

4 participants