diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-08-05 21:28:29 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-08-05 21:28:29 +0000 |
| commit | 3d40d5e70ebe21b7d52467987bffad8aea16f29b (patch) | |
| tree | b7fb0cd8a67b77ed297451481c956cba430aa3f2 /src/backend/executor/execAmi.c | |
| parent | c78248c91d5147a45907cb05d2c424cf4a3a792d (diff) | |
| download | postgresql-3d40d5e70ebe21b7d52467987bffad8aea16f29b.tar.gz postgresql-3d40d5e70ebe21b7d52467987bffad8aea16f29b.zip | |
Do not allow Unique nodes to be scanned backwards. The code claimed that it
would work, but in fact it didn't return the same rows when moving backwards
as when moving forwards. This would have no visible effect in a DISTINCT
query (at least assuming the column datatypes use a strong definition of
equality), but it gave entirely wrong answers for DISTINCT ON queries.
Diffstat (limited to 'src/backend/executor/execAmi.c')
| -rw-r--r-- | src/backend/executor/execAmi.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/backend/executor/execAmi.c b/src/backend/executor/execAmi.c index 154301a67f9..1381a4a4f09 100644 --- a/src/backend/executor/execAmi.c +++ b/src/backend/executor/execAmi.c @@ -6,7 +6,7 @@ * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/backend/executor/execAmi.c,v 1.96 2008/07/26 19:15:35 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/executor/execAmi.c,v 1.97 2008/08/05 21:28:29 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -414,9 +414,6 @@ ExecSupportsBackwardScan(Plan *node) case T_Sort: return true; - case T_Unique: - return ExecSupportsBackwardScan(outerPlan(node)); - case T_Limit: return ExecSupportsBackwardScan(outerPlan(node)); |