Re: FW: [HACKERS] decode() function for PostgreSQL

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Archana Bharatee <archana(at)ecomscience(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: FW: [HACKERS] decode() function for PostgreSQL
Date: 2002-04-11 05:35:39
Message-ID: 200204110535.g3B5ZdS08262@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


I have reviewed this patch. It actually outputs just a new query string
that replaces the DECODE with CASE, as you mentioned. I don't think
this has use in our existing code. It would be more appropriate if
somehow it converted the string in the gram.y file from DECODE to
CASE.

However, with CASE available and standard SQL, I am not even sure if we
would want to support DECODE, except for conversion usage. Was that the
purpose of the function you sent in?

Do other Oracle users see a need for something that understands DECODE?

---------------------------------------------------------------------------

Archana Bharatee wrote:
> Dear Bruce,
>
> I am not familiar with the process of applying patches. I am attaching the
> source-code for the decode function we have developed for PostgreSQL. Can
> you please do the needful, or redirect me to the appropriate person.
>
> Thanks a lot,
>
> warm regards,
> Archana Bharatee
>
> -----Original Message-----
> From: Bruce Momjian [mailto:pgman(at)candle(dot)pha(dot)pa(dot)us]
> Sent: Monday, February 25, 2002 10:33 PM
> To: Archana Bharatee
> Cc: pgsql-hackers(at)postgresql(dot)org
> Subject: Re: [HACKERS] decode()
>
>
> Archana Bharatee wrote:
> > We have developed a 'C' function to provide Oracle decode() functionality.
> > we have developed a C function DECODE which accepts an SQL Query and
> > replaces occurence of decode with case-when-then-else-end and returns the
> > query. There is no limitation of fixed number of parameters. This needs
> to
> > be added to PostgreSQL library. How do we do this?
>
> Send the patch to the patches list. A context diff against against 7.2
> or CVS is just fine.
>
> --
> Bruce Momjian | http://candle.pha.pa.us
> pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
> + If your life is a hard drive, | 830 Blythe Avenue
> + Christ can be your backup. | Drexel Hill, Pennsylvania 19026

[ Attachment, skipping... ]

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Stefan Lindner 2002-04-12 20:29:52 PostgreSQL 7.2.1 and Sun's C compiler under Solaris8
Previous Message Christopher Kings-Lynne 2002-04-10 07:09:18 BETWEEN SYMMETRIC/ASYMMETRIC patch