From: Tom Lane <tgl@sss.pgh.pa.us>
To: johnnnnnn <john@phaedrusdeinus.org>
Cc: pgsql-hackers@postgresql.org
Subject: Re: Upgrading the backend's error-message infrastructure
Date: Fri, 14 Mar 2003 12:23:04 -0500
Message-ID: <22063.1047662584@sss.pgh.pa.us> (raw)
In-Reply-To: <20030314161922.GG74817@performics.com>
References: <9168.1047588660@sss.pgh.pa.us>
<20030314161922.GG74817@performics.com>
johnnnnnn <john@phaedrusdeinus.org> writes:
> It would seem to make more sense to me to provide a different
> function(s) which allows the lookup Messages, Details, and Hints based
> on the SQLSTATE.
This would constrain us to have a different SQLSTATE for every error
message, which we aren't going to do. See elsewhere in thread. It's
also unclear how you insert parameters into error strings if you do this.
> - Less clutter and wasted space on the wire.
I am not really concerned about shaving bytes transmitted for an error
condition. If that's a performance-critical path for your app, you need
to rewrite the app ;-)
> - Removal of localization from error/notice generation libraries. This
> should make that section of code simpler and more fault-tolerant.
And you put it where, instead?
The existing scheme for localization works fine AFAICT. I don't have
any interest in reinventing it (nor any chance of getting this done for
7.4, if I were to try...)
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-interfaces@postgresql.org
Cc: tgl@sss.pgh.pa.us, john@phaedrusdeinus.org, pgsql-hackers@postgresql.org
Subject: Re: Upgrading the backend's error-message infrastructure
In-Reply-To: <22063.1047662584@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox