Sorry, I overlooked this when reviewing the previous version of the patch:
On 18.10.10 20:37, Tor Didriksen wrote:
> === modified file 'mysql-test/t/select.test'
> --- a/mysql-test/t/select.test 2010-09-09 15:00:33 +0000
> +++ b/mysql-test/t/select.test 2010-10-18 16:37:32 +0000
> @@ -4147,3 +4147,22 @@ DROP VIEW view_t1;
> DROP TABLE t1;
>
> --echo # End of test BUG#54515
> +
> +--echo #
> +--echo # Bug #57203 Assertion `field_length<= 255' failed.
> +--echo #
> +
> +SELECT coalesce((avg(distinct (geomfromtext("point(25379 -22010)")))))
> +UNION ALL
> +SELECT coalesce((avg(distinct (geomfromtext("point(25379 -22010)")))))
> +AS foo
> +;
> +
> +CREATE table t1(a text) engine=myisam;
The test case is SE-neutral, there is no point in explicit MyISAM
specification.
Otherwise the patch looks good to me.
