RE: Patch for Improved Syntax Error Reporting

From: Dave Page <dpage(at)vale-housing(dot)co(dot)uk>
To: "'Tom Lane'" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Dave Page <dpage(at)vale-housing(dot)co(dot)uk>
Cc: "'Bruce Momjian'" <pgman(at)candle(dot)pha(dot)pa(dot)us>, Neil Padgett <npadgett(at)redhat(dot)com>, pgsql-patches(at)postgresql(dot)org
Subject: RE: Patch for Improved Syntax Error Reporting
Date: 2001-08-02 21:59:57
Message-ID: 8568FC767B4AD311AC33006097BCD3D61A2D6E@woody.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

> -----Original Message-----
> From: Tom Lane [mailto:tgl(at)sss(dot)pgh(dot)pa(dot)us]
> Sent: 02 August 2001 22:16
> To: Dave Page
> Cc: 'Bruce Momjian'; Neil Padgett; pgsql-patches(at)postgresql(dot)org
> Subject: Re: [PATCHES] Patch for Improved Syntax Error Reporting
>
>
> Dave Page <dpage(at)vale-housing(dot)co(dot)uk> writes:
> > In the case of pgAdmin, it displays the error messages as
> returned by
> > the backend. I have no desire to start maintaining my own list of
> > error codes/messages, especially as those returned by the
> backend are
> > generally user friendly, and I certainly don't want to
> start parsing
> > user supplied SQL statements myself - I'd rather leave that to the
> > real parser.
>
> Maybe I'm confused, but I don't think any of the proposals
> would require you to do any of those things. The only case
> that requires any significant new effort from the client app
> is where it reformats queries before sending them on to the
> parser ... but that is knowledge you've already built into
> the client, no? You just have to remember what you did so
> that you can map a backend error index into the original text.

Like I said in another message, I seem to only have received bits of this
thread - I've probably misunderstood the comments that I read before writing
that.... it's been a long day and it's 11PM here now!

Regards, Dave.

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2001-08-02 22:11:40 Re: Patch for Improved Syntax Error Reporting
Previous Message Bruce Momjian 2001-08-02 21:59:25 Re: Patch for Improved Syntax Error Reporting