-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Remove GetSessionVars
in table.MutateContext
#54397
Labels
type/enhancement
The issue or PR belongs to an enhancement.
Comments
13 tasks
This was referenced Jul 4, 2024
13 tasks
13 tasks
13 tasks
This was referenced Jul 22, 2024
13 tasks
This was referenced Aug 5, 2024
13 tasks
13 tasks
13 tasks
13 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Enhancement
GetSessionVars
returnsvariable.SessionVars
which exposes too many fields and it's hard to build correctly. It's better to remove it and exposes some fields we really needed outside totable.MutateContext
.MutateBuffers
to manage buffers #54394MutateContext
#54433BaseKVEncoder
private and provide some wrapped methods #54673GetSessionVars
fromMutateContext
intables
#54715MutateContext
fromGetSessionVars()
#54767RowIDShardGenerator
andReservedRowIDAlloc
to alloc auto row id #54789CachedTableSupport
andTemporaryTableSupport
forMutateContext
#54900SkipWriteUntouchedIndices
forTable.UpdateRecord
#55192DupKeyCheckMode
for table mutations #55194DupKeyCheckLazy
to check duplicated key lazily #55246PessimisticLazyDupKeyCheckMode
to determine lazy mode in pessimistic txn #55360The text was updated successfully, but these errors were encountered: