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.

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

Bug1581949 5.6 #544

Merged
merged 2 commits into from
May 30, 2016
Merged

Conversation

laurynas-biveinis
Copy link
Contributor

@percona-ysorokin
Copy link
Collaborator

LGTM

…le owning threads execute new queries)

INFORMATION_SCHEMA.GLOBAL_TEMPORARY_TABLES query cloning temp table
handler passed the memory root of the temp table connection thread
object. This memory root is being freed at the end of query of that
thread, which may result, depending on scheduling, in the
GLOBAL_TEMPORARY_TABLES-querying thread accessing cloned object after it
has been freed.

Fixed by using a memory root of the GLOBAL_TEMPORARY_TABLES-querying
thread.

Add percona_show_temp_tables_debug testcase to test this exact scenario
and percona_show_temp_tables_stress to test querying
GLOBAL_TEMPORARY_TABLES in parallel to creating and destroying them.
@laurynas-biveinis
Copy link
Contributor Author

@percona-ysorokin
Copy link
Collaborator

Looks good

@laurynas-biveinis laurynas-biveinis merged commit 4834d54 into percona:5.6 May 30, 2016
@laurynas-biveinis laurynas-biveinis deleted the bug1581949-5.6 branch May 30, 2016 14:43
inikep pushed a commit to inikep/percona-server that referenced this pull request Apr 23, 2020
Summary:
Updated code to use DBUG_ENTER_FUNC instead of DBUG_ENTER("Function name") in storage/rocksdb
Closes facebook/mysql-5.6#544

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

Successfully ran ~/mysql/tools/mysqltest.sh --testset=RocksDBBig

Reviewed By: gunnarku

Differential Revision: D4609243

Pulled By: aartibasant

fbshipit-source-id: 2ab6166bfbf
inikep pushed a commit to inikep/percona-server that referenced this pull request Feb 24, 2021
Summary:
Updated code to use DBUG_ENTER_FUNC instead of DBUG_ENTER("Function name") in storage/rocksdb
Closes facebook/mysql-5.6#544

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

Successfully ran ~/mysql/tools/mysqltest.sh --testset=RocksDBBig

Reviewed By: gunnarku

Differential Revision: D4609243

Pulled By: aartibasant

fbshipit-source-id: 2ab6166bfbf
inikep pushed a commit to inikep/percona-server that referenced this pull request Nov 15, 2021
Summary:
Updated code to use DBUG_ENTER_FUNC instead of DBUG_ENTER("Function name") in storage/rocksdb
Closes facebook/mysql-5.6#544

Reviewed By: gunnarku

Differential Revision: D4609243 (facebook/mysql-5.6@4207a0c)

Pulled By: aartibasant

fbshipit-source-id: 6f106141675
ldonoso pushed a commit to ldonoso/percona-server that referenced this pull request Mar 15, 2022
Summary:
Updated code to use DBUG_ENTER_FUNC instead of DBUG_ENTER("Function name") in storage/rocksdb
Closes facebook/mysql-5.6#544

Reviewed By: gunnarku

Differential Revision: D4609243

Pulled By: aartibasant
inikep pushed a commit to inikep/percona-server that referenced this pull request Apr 9, 2024
Summary:
Updated code to use DBUG_ENTER_FUNC instead of DBUG_ENTER("Function name") in storage/rocksdb
Closes facebook/mysql-5.6#544

Reviewed By: gunnarku

Differential Revision: D4609243

Pulled By: aartibasant
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants