| From: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Hackers <pgsql-hackers(at)postgresql(dot)org>, pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: [HACKERS] Proposed GUC Variable |
| Date: | 2002-08-28 06:26:47 |
| Message-ID: | Pine.LNX.4.21.0208281618300.23917-102000@linuxworld.com.au |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches |
On Wed, 28 Aug 2002, Gavin Sherry wrote:
> On Tue, 27 Aug 2002, Tom Lane wrote:
>
> > Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > > But we should have some default to print some of the query,
> >
> > Why? So far you've been told by two different people (make that three
> > now) that such a behavior is useless, and no one's weighed in in its
> > favor ...
>
> I completely agree. Nothing wrong with adding another guc variable and
> since it is a debug variable people expect lots of verbosity.
Attached is the patch. debug_print_error_query is set to false by default.
For want of a better phrase, I've prepended 'original query: ' to the
error message to highlight why it is in the log.
Gavin
| Attachment | Content-Type | Size |
|---|---|---|
| guc.diff.gz | application/x-gzip | 1022 bytes |
| guc-doc.diff.gz | application/x-gzip | 316 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Christopher Kings-Lynne | 2002-08-28 06:53:46 | PREPARE, FK's and VIEWs |
| Previous Message | Marc Lavergne | 2002-08-28 04:39:03 | Re: C vs. C++ contributions |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Neil Conway | 2002-08-28 06:37:27 | Re: rules regression test fix |
| Previous Message | Joe Conway | 2002-08-28 04:54:37 | Anonymous-record-types omission |