mirror of
				https://github.com/postgres/postgres.git
				synced 2025-11-03 09:13:20 +03:00 
			
		
		
		
	Fix still another bug in commit 35fcb1b3d: it failed to fully initialize
the SortSupport states it introduced to allow the executor to re-check
ORDER BY expressions containing distance operators.  That led to a null
pointer dereference if the sortsupport code tried to use ssup_cxt.  The
problem only manifests in narrow cases, explaining the lack of previous
field reports.  It requires a GiST-indexable distance operator that lacks
SortSupport and is on a pass-by-ref data type, which among core+contrib
seems to be only btree_gist's interval opclass; and it requires the scan
to be done as an IndexScan not an IndexOnlyScan, which explains how
btree_gist's regression test didn't catch it.  Per bug #14134 from
Jihyun Yu.
Peter Geoghegan
Report: <20160511154904.2603.43889@wrigleys.postgresql.org>
		
	
		
			
				
	
	
		
			110 lines
		
	
	
		
			3.4 KiB
		
	
	
	
		
			Plaintext
		
	
	
	
	
	
			
		
		
	
	
			110 lines
		
	
	
		
			3.4 KiB
		
	
	
	
		
			Plaintext
		
	
	
	
	
	
-- interval check
 | 
						|
CREATE TABLE intervaltmp (a interval);
 | 
						|
\copy intervaltmp from 'data/interval.data'
 | 
						|
SET enable_seqscan=on;
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a <  '199 days 21:21:23';
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   329
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a <= '199 days 21:21:23';
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   330
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a  = '199 days 21:21:23';
 | 
						|
 count 
 | 
						|
-------
 | 
						|
     1
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a >= '199 days 21:21:23';
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   271
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a >  '199 days 21:21:23';
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   270
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT a, a <-> '199 days 21:21:23' FROM intervaltmp ORDER BY a <-> '199 days 21:21:23' LIMIT 3;
 | 
						|
                  a                  |               ?column?               
 | 
						|
-------------------------------------+--------------------------------------
 | 
						|
 @ 199 days 21 hours 21 mins 23 secs | @ 0
 | 
						|
 @ 183 days 6 hours 52 mins 48 secs  | @ 16 days 14 hours 28 mins 35 secs
 | 
						|
 @ 220 days 19 hours 5 mins 42 secs  | @ 21 days -2 hours -15 mins -41 secs
 | 
						|
(3 rows)
 | 
						|
 | 
						|
CREATE INDEX intervalidx ON intervaltmp USING gist ( a );
 | 
						|
SET enable_seqscan=off;
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a <  '199 days 21:21:23'::interval;
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   329
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a <= '199 days 21:21:23'::interval;
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   330
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a  = '199 days 21:21:23'::interval;
 | 
						|
 count 
 | 
						|
-------
 | 
						|
     1
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a >= '199 days 21:21:23'::interval;
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   271
 | 
						|
(1 row)
 | 
						|
 | 
						|
SELECT count(*) FROM intervaltmp WHERE a >  '199 days 21:21:23'::interval;
 | 
						|
 count 
 | 
						|
-------
 | 
						|
   270
 | 
						|
(1 row)
 | 
						|
 | 
						|
EXPLAIN (COSTS OFF)
 | 
						|
SELECT a, a <-> '199 days 21:21:23' FROM intervaltmp ORDER BY a <-> '199 days 21:21:23' LIMIT 3;
 | 
						|
                                QUERY PLAN                                 
 | 
						|
---------------------------------------------------------------------------
 | 
						|
 Limit
 | 
						|
   ->  Index Only Scan using intervalidx on intervaltmp
 | 
						|
         Order By: (a <-> '@ 199 days 21 hours 21 mins 23 secs'::interval)
 | 
						|
(3 rows)
 | 
						|
 | 
						|
SELECT a, a <-> '199 days 21:21:23' FROM intervaltmp ORDER BY a <-> '199 days 21:21:23' LIMIT 3;
 | 
						|
                  a                  |               ?column?               
 | 
						|
-------------------------------------+--------------------------------------
 | 
						|
 @ 199 days 21 hours 21 mins 23 secs | @ 0
 | 
						|
 @ 183 days 6 hours 52 mins 48 secs  | @ 16 days 14 hours 28 mins 35 secs
 | 
						|
 @ 220 days 19 hours 5 mins 42 secs  | @ 21 days -2 hours -15 mins -41 secs
 | 
						|
(3 rows)
 | 
						|
 | 
						|
SET enable_indexonlyscan=off;
 | 
						|
EXPLAIN (COSTS OFF)
 | 
						|
SELECT a, a <-> '199 days 21:21:23' FROM intervaltmp ORDER BY a <-> '199 days 21:21:23' LIMIT 3;
 | 
						|
                                QUERY PLAN                                 
 | 
						|
---------------------------------------------------------------------------
 | 
						|
 Limit
 | 
						|
   ->  Index Scan using intervalidx on intervaltmp
 | 
						|
         Order By: (a <-> '@ 199 days 21 hours 21 mins 23 secs'::interval)
 | 
						|
(3 rows)
 | 
						|
 | 
						|
SELECT a, a <-> '199 days 21:21:23' FROM intervaltmp ORDER BY a <-> '199 days 21:21:23' LIMIT 3;
 | 
						|
                  a                  |               ?column?               
 | 
						|
-------------------------------------+--------------------------------------
 | 
						|
 @ 199 days 21 hours 21 mins 23 secs | @ 0
 | 
						|
 @ 183 days 6 hours 52 mins 48 secs  | @ 16 days 14 hours 28 mins 35 secs
 | 
						|
 @ 220 days 19 hours 5 mins 42 secs  | @ 21 days -2 hours -15 mins -41 secs
 | 
						|
(3 rows)
 | 
						|
 |