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

remove stmtctx dependency from tablecodec and rowcodec #48751

Closed
Tracked by #47355
YangKeao opened this issue Nov 21, 2023 · 0 comments · Fixed by #48816
Closed
Tracked by #47355

remove stmtctx dependency from tablecodec and rowcodec #48751

YangKeao opened this issue Nov 21, 2023 · 0 comments · Fixed by #48816
Assignees
@YangKeao
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@YangKeao
Copy link
Member

Enhancement

Just like what we have done in the util/codec, the dependency stmtctx can also be removed from tablecodec and rowcodec. It will make the logical dependency of these functions in tablecodec and rowcodec much clearer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@YangKeao YangKeao

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.

util/rowcodec,tablecodec: remove stmtctx dependency from rowcodec and tablecodec YangKeao/tidb
1 participant
@YangKeao