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. ---

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 variant

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 and approves

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 pgman@candle.pha.pa.us writes:

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-25 Thread Bruce Momjian
Tom Lane wrote: Bruce Momjian pgman@candle.pha.pa.us 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

[PATCHES] TODO item - tid operator

2005-10-24 Thread Mark Kirkwood
Please find enclosed a patch that adds a '' operator for tid types (per TODO item). Patch is against current sources (beta4). If I have understood the requirement properly, the relevant function 'tidne' already existed in src/backend/utils/adt/tid.c, just needed an ifdef NOT_USED removed. The

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 ==

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