[Maria-developers] MDEV-10764 PL/SQL parser - Phase 2 / MDEV-12137
Hello everyone, I've done a patch for MDEV-12137 based on branch bb-10.2-ext (DELETE statement with the same source and target). At this time the patch is for all sql_mode. It could be interesting to add an information in explain plan to indicate the extra step. Something like : +------+--------------------+------------+------+---------------+---------------+---------+--------------------+------+-------------+ | id | select_type | table | type | possible_keys | key | key_len | ref | rows | Extra | +------+--------------------+------------+------+---------------+---------------+---------+--------------------+------+-------------+ | 1 | DELETE | tab_delete | RND | NULL | NULL | NULL | NULL | 1 | Using rowid | | 1 | PRIMARY | tab_delete | ALL | NULL | NULL | NULL | NULL | 1 | Using where | | 2 | DEPENDENT SUBQUERY | b | ref | tab_delete_c1 | tab_delete_c1 | 4 | test.tab_delete.c1 | 1 | Using index | +------+--------------------+------------+------+---------------+---------------+---------+--------------------+------+-------------+ Can anyone review it ? Best regards, Jérôme.
Hello Jerome, Sergei Golubchik will review your patch. Thanks! On 05/05/2017 07:24 PM, jerome brauge wrote:
Hello everyone, I've done a patch for MDEV-12137 based on branch bb-10.2-ext (DELETE statement with the same source and target). At this time the patch is for all sql_mode. It could be interesting to add an information in explain plan to indicate the extra step. Something like : +------+--------------------+------------+------+---------------+---------------+---------+--------------------+------+-------------+ | id | select_type | table | type | possible_keys | key | key_len | ref | rows | Extra | +------+--------------------+------------+------+---------------+---------------+---------+--------------------+------+-------------+ | 1 | DELETE | tab_delete | RND | NULL | NULL | NULL | NULL | 1 | Using rowid | | 1 | PRIMARY | tab_delete | ALL | NULL | NULL | NULL | NULL | 1 | Using where | | 2 | DEPENDENT SUBQUERY | b | ref | tab_delete_c1 | tab_delete_c1 | 4 | test.tab_delete.c1 | 1 | Using index | +------+--------------------+------------+------+---------------+---------------+---------+--------------------+------+-------------+
Can anyone review it ?
Best regards, Jérôme.
participants (2)
-
Alexander Barkov
-
jerome brauge