Re: [Maria-developers] [Commits] 3bbf3d2: MDEV-10216: Assertion `strcmp(share->unique_file_name, filename) ||
![](https://secure.gravatar.com/avatar/32b3f5fe52f3e988b4cd6460e7f68d91.jpg?s=120&d=mm&r=g)
Hi Nirbhay, Looks good, thanks! On Fri, Jul 01, 2016 at 02:47:29PM -0400, Nirbhay Choubey wrote:
revision-id: 3bbf3d2e246da1a2fd458b634f30cbe35afb5668 (mariadb-10.2.0-142-g3bbf3d2) parent(s): 76f492e26d8d45c678c570be2cf4ca5d238edceb author: Nirbhay Choubey committer: Nirbhay Choubey timestamp: 2016-07-01 14:47:27 -0400 message:
MDEV-10216: Assertion `strcmp(share->unique_file_name,filename) ||
.. share->last_version' failed in myisam/mi_open.c:67: test_if_reopen
During the RENAME operation since the renamed temporary table is also opened and added to myisam_open_list/maria_open_list, resetting the last_version at the end of operation (HA_EXTRA_PREPARE_FOR_RENAME) will cause an assertion failure when a subsequent query tries to open an additional temporary table instance and thus attempts to reuse it from the open table list.
This commit fixes the issue by skipping flush/close operations executed toward the end of ALTER for temporary tables. It also enables a shortcut for simple ALTERs (like rename, disable/enable keys) on temporary tables.
As safety checks, added some assertions at code points that should not be hit for temporary tables. ...
Regards, Sergey
![](https://secure.gravatar.com/avatar/0c65dc0ac6187abaea3a690ac1c1ee5b.jpg?s=120&d=mm&r=g)
Hi Monty!
Can you please 2nd review this patch?
http://lists.askmonty.org/pipermail/commits/2016-July/009523.html
Best,
Nirbhay
On Tue, Jul 5, 2016 at 4:51 AM, Sergey Vojtovich
Hi Nirbhay,
Looks good, thanks!
On Fri, Jul 01, 2016 at 02:47:29PM -0400, Nirbhay Choubey wrote:
revision-id: 3bbf3d2e246da1a2fd458b634f30cbe35afb5668 (mariadb-10.2.0-142-g3bbf3d2) parent(s): 76f492e26d8d45c678c570be2cf4ca5d238edceb author: Nirbhay Choubey committer: Nirbhay Choubey timestamp: 2016-07-01 14:47:27 -0400 message:
MDEV-10216: Assertion `strcmp(share->unique_file_name,filename) ||
.. share->last_version' failed in myisam/mi_open.c:67: test_if_reopen
During the RENAME operation since the renamed temporary table is also opened and added to myisam_open_list/maria_open_list, resetting the last_version at the end of operation (HA_EXTRA_PREPARE_FOR_RENAME) will cause an assertion failure when a subsequent query tries to open an additional temporary table instance and thus attempts to reuse it from the open table list.
This commit fixes the issue by skipping flush/close operations executed toward the end of ALTER for temporary tables. It also enables a shortcut for simple ALTERs (like rename, disable/enable keys) on temporary tables.
As safety checks, added some assertions at code points that should not be hit for temporary tables. ...
Regards, Sergey _______________________________________________ commits mailing list commits@mariadb.org https://lists.askmonty.org/cgi-bin/mailman/listinfo/commits
participants (2)
-
Nirbhay Choubey
-
Sergey Vojtovich