On Mon, 23 Sep 2013 10:27:28 +0400, Vlad Lesin wrote:
>> - I guess changes in sql_prepare.cc are no longer required and can be
>> reverted?
> No, they are still required because objects of "Item" class are
> allocated during parsing and thd->free_list contains list of those objects.
>
OK, I see it now. Shouldn't we instead of adjusting the assertion in
sql_prepare.cc just reset thd->free_list after calling
sql_set_variables() in mysql_execute_command()? It was impossible with
the previous implementation, but is possible now, right?
Hi Vlad,
On Mon, 23 Sep 2013 10:27:28 +0400, Vlad Lesin wrote:
>> - I guess changes in sql_prepare.cc are no longer required and can be
>> reverted?
> No, they are still required because objects of "Item" class are
> allocated during parsing and thd->free_list contains list of those objects.
>
OK, I see it now. Shouldn't we instead of adjusting the assertion in command( )? It was impossible with
sql_prepare.cc just reset thd->free_list after calling
sql_set_variables() in mysql_execute_
the previous implementation, but is possible now, right?