| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | "Pavel Stehule" <pavel(dot)stehule(at)hotmail(dot)com> |
| Cc: | pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: SQL/XML publishing function experimental patch |
| Date: | 2005-09-12 14:05:22 |
| Message-ID: | 21127.1126533922@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
"Pavel Stehule" <pavel(dot)stehule(at)hotmail(dot)com> writes:
> This patch contains SQL/XML public function XMLFOREST, XMLELEMENT, ... based
> on CString type. This patch hasn't high quality, but can be usefull for
> testing funcionality, and maybe for some people.
1. Why did you base the datatype on cstring? That's inappropriate for
anything except guaranteed-short strings, because it's not toastable.
2. Is it really necessary to hack up the grammar for this? Seems like
you should create ordinary functions to manipulate the datatype(s).
If every datatype thought it could do this sort of thing, we'd have
a completely unmanageable grammar.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2005-09-12 14:27:24 | Re: SQL/XML publishing function experimental patch |
| Previous Message | Michael Meskes | 2005-09-12 12:36:34 | Re: [HACKERS] ECPG ignores SAVEPOINT if first statement of a transaction |