| From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | "Henshall, Stuart - WCP" <SHenshall(at)westcountrypublications(dot)co(dot)uk>, "'pgsql-bugs(at)postgresql(dot)org'" <pgsql-bugs(at)postgresql(dot)org> |
| Subject: | Re: RAISE concatination/variables in plpgsql |
| Date: | 2001-05-08 00:08:26 |
| Message-ID: | 200105080008.f4808Q813238@candle.pha.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> >> Looking at the plpgsql code, it's clear that what's actually implemented
> >> is
> >> RAISE level string-literal [ , variable [ , variable [ ... ] ]
>
> > I see the current docs showing:
>
> > RAISE level 'format' [, identifier [...]];
>
> > Is this acceptable? Should 'identifier' be 'variable'?
>
> Probably. And 'format' is even more misleading, since it implies that
> you write a printf-like format string, which you do not. The output is
> just the concatenation of the literal and the variable values.
Updated with patch attached.
>
> >> which is pretty bletcherous; seems like it should accept a list of
> >> expressions instead. But for 7.1, I guess this is a documentation bug
> >> rather than something to change in the code.
>
> > Do I need a TODO item here?
>
> It seems in need of fixing to me ...
TODO updated with:
* Allow Pl/PgSQL's RAISE function to take expressions
--
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 | Content-Type | Size |
|---|---|---|
| unknown_filename | text/plain | 1.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2001-05-08 00:15:22 | Re: RAISE concatination/variables in plpgsql |
| Previous Message | Tom Lane | 2001-05-08 00:05:17 | Re: backend dies on 7.1.1 loading large datamodel. |