Re: Hmm, nodeUnique doesn't really support backwards scan too well

From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-bugs(at)postgreSQL(dot)org
Subject: Re: Hmm, nodeUnique doesn't really support backwards scan too well
Date: 2008-08-05 22:03:32
Message-ID: 1217973812.4549.112.camel@ebony.2ndQuadrant
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs


On Tue, 2008-08-05 at 13:07 -0400, Tom Lane wrote:

> We could probably fix this by complicating the logic in ExecUnique,
> but I wonder whether it wouldn't be better to just stop treating
> Unique nodes as backwards-scannable.

No problem there.

> The only reason for that
> node type to exist (as opposed to using Group nodes) is that it's
> simple and low-overhead. So complicating it to support a corner case
> that no one has noticed in many years might be counterproductive.
> Thoughts?

I've never seen anyone scan backwards like this at all in practical use.

I knew it was possible, but never seen it done.

It seems entirely probable nobody else has either. It's a PostgreSQL
extension, so people arriving from outside don't even know it exists,
plus its always had bugs so those in-the-know don't use it either:
http://archives.postgresql.org/pgsql-bugs/1998-06/msg00049.php

My perceptions may not match others...

--
Simon Riggs www.2ndQuadrant.com
PostgreSQL Training, Services and Support

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2008-08-05 22:40:08 Re: Hmm, nodeUnique doesn't really support backwards scan too well
Previous Message Tom Lane 2008-08-05 17:07:11 Hmm, nodeUnique doesn't really support backwards scan too well