mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Dept of second thoughts: the IQ of estimate_array_length() needs to be

Browse Source 
kept on par with that of scalararraysel(), else estimates that should
track might not.  Hence teach it about binary-compatible cases, too.
This commit is contained in:
Tom Lane 2007-01-28 02:53:34 +00:00
parent c2c0b318d8
commit a053437d9e
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/backend/utils/adt/selfuncs.c
View File

@ -15,7 +15,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/utils/adt/selfuncs.c,v 1.222 2007/01/28 01:37:38 tgl Exp $ * $PostgreSQL: pgsql/src/backend/utils/adt/selfuncs.c,v 1.223 2007/01/28 02:53:34 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -1698,6 +1698,9 @@ scalararraysel(PlannerInfo *root,
int int
estimate_array_length(Node *arrayexpr) estimate_array_length(Node *arrayexpr)
{ {
/* look through any binary-compatible relabeling of arrayexpr */
arrayexpr = strip_array_coercion(arrayexpr);
if (arrayexpr && IsA(arrayexpr, Const)) if (arrayexpr && IsA(arrayexpr, Const))
{ {
Datum arraydatum = ((Const *) arrayexpr)->constvalue; Datum arraydatum = ((Const *) arrayexpr)->constvalue;