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

Update more in-memory selects #41254

Closed
wants to merge 1 commit into from
Closed

Update more in-memory selects #41254

wants to merge 1 commit into from

Conversation

Hamms
Copy link
Contributor

@Hamms Hamms commented Jun 22, 2021

Follow-up to #41252

I don't know whether or not any of these are currently actively causing any significant performance issues, but they're all less performant than they should be.

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

Follow-up to #41252

I don't know whether or not any of these are currently actively causing any significant performance issues, but they're all less performant than they should be.
Base automatically changed from speed-up-unplugged-lesson to staging June 23, 2021 02:01
@Hamms Hamms closed this Jun 29, 2021
@Hamms Hamms deleted the fewer-in-memory-selects branch June 29, 2021 17:19
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.

None yet

1 participant