-
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
do not allow historical read in local temporary tables #25938
Comments
/assign |
It can be closed by #27270. @djshow832 details are as follows: for _, query := range queries {
localSQL := genLocalTemporarySQL(query.sql)
queries = append(queries, struct{ sql string }{sql: localSQL})
} and tk.MustExec("set @@tidb_snapshot=NOW(6)")
for _, query := range queries {
// Will success here for compatibility with some tools like dumping
rs := tk.MustQuery(query.sql)
rs.Check(testkit.Rows())
} |
@sylzd I think we should forbid history read on local temporary table. This behavior is different from global temporary tables which need to be compatible with some tools like dumpling. |
Got it~ local temporary tables do not need adapt to dumpling. |
No description provided.
The text was updated successfully, but these errors were encountered: