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

Disable plan cache for all inserts for vtgate and vttablet. #5471

Closed

Conversation

aquarapid
Copy link
Contributor

These blow out memory during bulk imports, and are not useful anyway.

Signed-off-by: Jacques Grove aquarapid@gmail.com

@aquarapid
Copy link
Contributor Author

Few WIP items:

  • tests
  • What are the stats impacts? Potentially just bypass large inserts?

@aquarapid
Copy link
Contributor Author

@systay can you review this approach, or suggest an alternative?

aquarapid and others added 2 commits February 27, 2020 19:29
These blow out memory during bulk imports, and are not useful anyway.

Signed-off-by: Jacques Grove <aquarapid@gmail.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Jacques Grove <aquarapid@gmail.com>
Signed-off-by: Jacques Grove <aquarapid@gmail.com>
@deepthi
Copy link
Member

deepthi commented Jun 8, 2021

Can we close this? The new caching should work well even for bulk inserts.

@aquarapid
Copy link
Contributor Author

Since the plan cache is now memory limited; we don't need this anymore

@aquarapid aquarapid closed this Jul 13, 2021
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