Re: Review: listagg aggregate
Pavel Stehule <pavel.stehule@gmail.com>
From: Pavel Stehule <pavel.stehule@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>, Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>, "David E. Wheeler" <david@kineticode.com>, "pgsql-hackers@postgresql.org Hackers" <pgsql-hackers@postgresql.org>
Date: 2010-01-28T17:19:59Z
Lists: pgsql-hackers
2010/1/28 Tom Lane <tgl@sss.pgh.pa.us>: > Pavel Stehule <pavel.stehule@gmail.com> writes: >> in 99.99% the second argument will be a constant. Can we use this >> information and optimize function for this case? > >> The detoast on every row can take some percent from a performance. > > What detoast? There won't be one for a constant, nor even for a > variable in any sane situation --- who's going to be using > multi-kilobyte delimiter values? And if they do, aren't they likely > to run out of memory for the result long before the repeated detoasts > become an interesting problem? You're arguing about a case that > seems quite irrelevant to the real world. > I asking with get_fn_expr_arg_stable() we are able to fix second parameter without some performance issues. Regards Pavel > regards, tom lane >