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

backport: Add subject filters in schema relation delete to force use of the index #2136

Merged
merged 2 commits into from
Nov 18, 2024

Conversation

vroldanbet
Copy link
Contributor

@vroldanbet vroldanbet commented Nov 18, 2024

This should allow for much faster queries under the transaction. We observed some customers reporting timeouts when writting a new schema and dropping a relation because it ended up in full table scans.

I also had to backport #2117 for MySQL tests to pass

This should allow for much faster queries under the transaction. We
observed some customers reporting timeouts when writting a new
schema and dropping a relation because it ended up in full table
scans.
@vroldanbet vroldanbet requested a review from a team as a code owner November 18, 2024 13:29
@vroldanbet vroldanbet self-assigned this Nov 18, 2024
@github-actions github-actions bot added the area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) label Nov 18, 2024
Seriously, it broke due to the DST change because it just *happen* to match the -5 hours window used for the test.

The fix is to move the test that changes the instance's timezone to its own, isolated, MySQL so it doesn't interfere with the other tests
@github-actions github-actions bot added the area/datastore Affects the storage system label Nov 18, 2024
@vroldanbet vroldanbet merged commit 158e886 into release-v1.38 Nov 18, 2024
22 checks passed
@vroldanbet vroldanbet deleted the backport-write-schema-fix branch November 18, 2024 14:18
@github-actions github-actions bot locked and limited conversation to collaborators Nov 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/datastore Affects the storage system area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants