diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2010-03-25 15:50:10 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2010-03-25 15:50:10 +0000 |
| commit | 5ceb13c8a12320a44fa211cf0ba32b1641270aeb (patch) | |
| tree | 3bfb94f86861aa7e22bc93f98ece4e419ec14220 | |
| parent | be8cebc717ef0b583ada09da8d0245793697c1a8 (diff) | |
| download | postgresql-5ceb13c8a12320a44fa211cf0ba32b1641270aeb.tar.gz postgresql-5ceb13c8a12320a44fa211cf0ba32b1641270aeb.zip | |
Fix ginint4_queryextract() to actually do what it was intended to do for an
unsatisfiable query, such as indexcol && empty_array. It should return -1
to tell GIN no scan is required; but silly typo disabled the logic for that,
resulting in unnecessary "GIN indexes do not support whole-index scans" error.
Per bug report from Jeff Trout.
Back-patch to 8.3 where the logic was introduced.
| -rw-r--r-- | contrib/intarray/_int_gin.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/contrib/intarray/_int_gin.c b/contrib/intarray/_int_gin.c index 2bf8fc54bb4..375ff3ae25e 100644 --- a/contrib/intarray/_int_gin.c +++ b/contrib/intarray/_int_gin.c @@ -1,5 +1,5 @@ /* - * $PostgreSQL: pgsql/contrib/intarray/_int_gin.c,v 1.10 2009/06/11 14:48:51 momjian Exp $ + * $PostgreSQL: pgsql/contrib/intarray/_int_gin.c,v 1.11 2010/03/25 15:50:10 tgl Exp $ */ #include "postgres.h" @@ -65,7 +65,7 @@ ginint4_queryextract(PG_FUNCTION_ARGS) } } - if (nentries == 0) + if (*nentries == 0) { switch (strategy) { |