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.

Sign up for GitHub

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

Jump to bottom

RCCheckTS should be managed by RC transaction context provider. #35497

Closed
Tracked by #30535
SpadeA-Tang opened this issue Jun 20, 2022 · 0 comments · Fixed by #35554
Closed
Tracked by #30535

RCCheckTS should be managed by RC transaction context provider. #35497

SpadeA-Tang opened this issue Jun 20, 2022 · 0 comments · Fixed by #35554
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@SpadeA-Tang
Copy link
Contributor

Enhancement

RCCheckTS, which is a field of StatementContext, is a transaction related variable, but it is initialized in ResetContextOfStmt currently. We should manager the modification of it within RC transaction context provider and let it read-only in StatementContext.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

txn: manage the initialization of RCCheckTS by transaction context provider SpadeA-Tang/tidb
1 participant
@SpadeA-Tang