Skip to content

sqlliveness: migrate to rbr compatible index#98463

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
jeffswenson:jeffswenson-rbr-sqlliveness-migration
Mar 14, 2023
Merged

sqlliveness: migrate to rbr compatible index#98463
craig[bot] merged 1 commit intocockroachdb:masterfrom
jeffswenson:jeffswenson-rbr-sqlliveness-migration

Conversation

@jeffswenson
Copy link
Copy Markdown
Collaborator

@jeffswenson jeffswenson commented Mar 12, 2023

Migrate sqlliveness to the regional by row compatible index. The version
gates are intended to follow the protocol discussed in the comment at
the top of upgrades/system_rbr_indexes.go.

As part of this change, the legacy session id encoding is treated as
belonging to the enum.One region. Treating the legacy encoding as
belonging to enum.One simplifies the migration as it is not necessary to
wait for old sessions to expire or clear old sessions from the jobs
table.

Part of #94843

Release note: none

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@jeffswenson jeffswenson force-pushed the jeffswenson-rbr-sqlliveness-migration branch from 6a61b91 to 21193fb Compare March 12, 2023 20:49
@jeffswenson jeffswenson changed the title Jeffswenson rbr sqlliveness migration sqlliveness: migrate to rbr compatible index Mar 12, 2023
@jeffswenson jeffswenson force-pushed the jeffswenson-rbr-sqlliveness-migration branch 2 times, most recently from 2be4e1f to ac8948c Compare March 12, 2023 22:51
@jeffswenson jeffswenson force-pushed the jeffswenson-rbr-sqlliveness-migration branch 4 times, most recently from be49347 to aca3d96 Compare March 13, 2023 16:17
@jeffswenson jeffswenson marked this pull request as ready for review March 13, 2023 16:18
@jeffswenson jeffswenson requested a review from a team March 13, 2023 16:18
@jeffswenson jeffswenson requested review from a team as code owners March 13, 2023 16:18
@jeffswenson jeffswenson force-pushed the jeffswenson-rbr-sqlliveness-migration branch from aca3d96 to 076f30e Compare March 13, 2023 18:11
Copy link
Copy Markdown
Contributor

@ajwerner ajwerner left a comment

Choose a reason for hiding this comment

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

:lgtm_strong:

Reviewed 22 of 22 files at r2, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained

Migrate sqlliveness to the regional by row compatible index. The version
gates are intended to follow the protocol discussed in the comment at
the top of upgrades/system_rbr_indexes.go.

As part of this change, the legacy session id encoding is treated as
belonging to the enum.One region. Treating the legacy encoding as
belonging to enum.One simplifies the migration as it is not necessary to
wait for old sessions to expire or clear old sessions from the jobs
table.

Part of cockroachdb#94843

Release note: none
@jeffswenson jeffswenson force-pushed the jeffswenson-rbr-sqlliveness-migration branch from 076f30e to 6cd6962 Compare March 13, 2023 21:28
@jeffswenson
Copy link
Copy Markdown
Collaborator Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 13, 2023

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 14, 2023

Build succeeded:

@craig craig bot merged commit 74f993a into cockroachdb:master Mar 14, 2023
@jeffswenson jeffswenson deleted the jeffswenson-rbr-sqlliveness-migration branch March 14, 2023 12:40
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.

3 participants