Re: [PATCHES] TODO item - tid <> operator

2006-06-14 Thread Bruce Momjian
Neil Conway wrote: > On Wed, 2006-06-14 at 18:08 -0400, Bruce Momjian wrote: > > Your patch has been added to the PostgreSQL unapplied patches list at: > > > > http://momjian.postgresql.org/cgi-bin/pgpatches > > > > It will be applied as soon as one of the PostgreSQL committers reviews > > an

Re: [PATCHES] TODO item - tid <> operator

2006-06-14 Thread Neil Conway
On Wed, 2006-06-14 at 18:08 -0400, Bruce Momjian wrote: > Your patch has been added to the PostgreSQL unapplied patches list at: > > http://momjian.postgresql.org/cgi-bin/pgpatches > > It will be applied as soon as one of the PostgreSQL committers reviews > and approves it. I applied a var

Re: [PATCHES] TODO item - tid <> operator

2006-06-14 Thread Bruce Momjian
Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. --- Ma

Re: [PATCHES] TODO item - tid <> operator

2005-10-26 Thread Bruce Momjian
This has been saved for the 8.2 release: http://momjian.postgresql.org/cgi-bin/pgpatches_hold --- Mark Kirkwood wrote: > Bruce Momjian wrote: > > Tom Lane wrote: > > > >>Bruce Momjian writes: > >> > >>>This has be

Re: [PATCHES] TODO item - tid <> operator

2005-10-25 Thread Mark Kirkwood
Bruce Momjian wrote: Tom Lane wrote: Bruce Momjian writes: This has been saved for the 8.2 release: http://momjian.postgresql.org/cgi-bin/pgpatches_hold Uh, why do we need this at all? "NOT (tid = tid)" covers the functionality already. tid should be a fully functional type, at

Re: [PATCHES] TODO item - tid <> operator

2005-10-25 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian writes: > > This has been saved for the 8.2 release: > > http://momjian.postgresql.org/cgi-bin/pgpatches_hold > > Uh, why do we need this at all? "NOT (tid = tid)" covers the > functionality already. tid should be a fully functional type, at least for = and !

Re: [PATCHES] TODO item - tid <> operator

2005-10-25 Thread Tom Lane
Bruce Momjian writes: > This has been saved for the 8.2 release: > http://momjian.postgresql.org/cgi-bin/pgpatches_hold Uh, why do we need this at all? "NOT (tid = tid)" covers the functionality already. I disagree strongly with renumbering existing hand-assigned OIDs for this. There's t

Re: [PATCHES] TODO item - tid <> operator

2005-10-25 Thread Bruce Momjian
This has been saved for the 8.2 release: http://momjian.postgresql.org/cgi-bin/pgpatches_hold --- Mark Kirkwood wrote: > Neil Conway wrote: > > > > > You also probably want to add the declaration for tidne() to >

Re: [PATCHES] TODO item - tid <> operator

2005-10-24 Thread Mark Kirkwood
Neil Conway wrote: You also probably want to add the declaration for tidne() to include/utils/builtins.h Doh! - Indeed, I obviously missed a few extra compile warnings! Revised patch attached. Index: src/backend/utils/adt/tid.c ===

Re: [PATCHES] TODO item - tid <> operator

2005-10-24 Thread Neil Conway
On Mon, 2005-24-10 at 20:09 +1300, Mark Kirkwood wrote: > *** src/backend/utils/adt/tid.c 27 May 2005 00:57:49 - 1.49 > --- src/backend/utils/adt/tid.c 24 Oct 2005 02:34:51 - > *** > *** 174,180 > arg1->ip_posid == arg2->ip_