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

3.x: Invalidate tablets on schema changes #382

Merged

Conversation

Bouncheck
Copy link
Collaborator

@Bouncheck Bouncheck commented Nov 26, 2024

Remove tablet mappings on select events

Registers an event listener dedicated to updating the TabletMap
held by Metadata. It will trigger removal of relevant mappings whenever it's
notified of any change to the table schema or keyspace schema including
dropping them.

Introduces an integration test that verifies this listener's behaviour.

@Bouncheck Bouncheck marked this pull request as ready for review November 26, 2024 18:11
@Bouncheck Bouncheck changed the title (WIP) 3.x: Invalidate tablets on schema changes 3.x: Invalidate tablets on schema changes Nov 26, 2024
@Bouncheck Bouncheck self-assigned this Nov 26, 2024
Registers an event listener dedicated to updating the `TabletMap`
held by `Metadata`. It will trigger removal of relevant mappings whenever it's
notified of any change to the table schema or keyspace schema including
dropping them.

Introduces an integration test that verifies this listener's behaviour.

Addresses scylladb#378.
@dkropachev
Copy link
Collaborator

@Bouncheck , 6.2.1 failed by reason not connected to this PR

@Bouncheck
Copy link
Collaborator Author

Bouncheck commented Nov 27, 2024

@dkropachev i need reapproval to merge
edit: nevermind, i went ahead since it was a formality by now

@Bouncheck Bouncheck merged commit 95fa50b into scylladb:scylla-3.x Nov 27, 2024
10 of 11 checks passed
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.

2 participants