Re: pl/pgsql - code review + question

From: "Richard Huxton" <dev(at)archonet(dot)com>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "Gary Stainburn" <gary(dot)stainburn(at)ringways(dot)co(dot)uk>, "pgsql-sql" <pgsql-sql(at)postgresql(dot)org>, "Josh Berkus" <josh(at)agliodbs(dot)com>
Subject: Re: pl/pgsql - code review + question
Date: 2001-07-19 09:04:30
Message-ID: 007001c11031$d339b160$1001a8c0@archonet.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-sql

From: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>

> "Richard Huxton" <dev(at)archonet(dot)com> writes:
> > Don't know why the parser for RAISE doesn't like string concat.
>
> Laziness ;-). Someone should fix plpgsql so that RAISE does take
> expressions, not just literals and simple variables. It probably
> wouldn't be a big change, but I've no time to look at it myself;
> any volunteers out there?

False Laziness perhaps (spot my Perl background)

OK - stick me down for having a look at it. Had a quick peek and I think
it's within my abilities. Give me a couple of weeks, because I haven't
looked at a YACC file since my university days (gram.y is YACC isn't it?)

I'll read up the rules for submitting patches and get something workable
back by August 3rd.

Josh - if I try and do OFFSET at the same time (presumably it's the same
change) do you fancy acting as a sanity test site?

- Richard Huxton

In response to

Responses

Browse pgsql-sql by date

  From Date Subject
Next Message Magnus Landahl 2001-07-19 09:04:40 Get the tables names?
Previous Message A. Mannisto 2001-07-19 08:57:37 Searching multiple fields from multiple tables