pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Started by Tom Laneabout 25 years ago8 messages
#1Tom Lane
tgl@postgresql.org

Date: Sunday, October 22, 2000 @ 19:32:45
Author: tgl

Update of /home/projects/pgsql/cvsroot/pgsql/src/test/regress/expected
from hub.org:/home/projects/pgsql/tmp/cvs-serv80691/src/test/regress/expected

Modified Files:
plpgsql.out inet.out foreign_key.out errors.out

----------------------------- Log Message -----------------------------

Some small polishing of Mark Hollomon's cleanup of DROP command: might
as well allow DROP multiple INDEX, RULE, TYPE as well. Add missing
CommandCounterIncrement to DROP loop, which could cause trouble otherwise
with multiple DROP of items affecting same catalog entries. Try to
bring a little consistency to various error messages using 'does not exist',
'nonexistent', etc --- I standardized on 'does not exist' since that's
what the vast majority of the existing uses seem to be.

#2Thomas Lockhart
lockhart@alumni.caltech.edu
In reply to: Tom Lane (#1)
Re: pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Modified Files:

...

Some small polishing of Mark Hollomon's cleanup of DROP command: might
as well allow DROP multiple INDEX, RULE, TYPE as well. Add missing
CommandCounterIncrement to DROP loop, which could cause trouble otherwise
with multiple DROP of items affecting same catalog entries. Try to
bring a little consistency to various error messages using 'does not exist',
'nonexistent', etc --- I standardized on 'does not exist' since that's
what the vast majority of the existing uses seem to be.

Good idea(s). Thanks for cleaning up the error messages...

- Thomas

#3Bruce Momjian
pgman@candle.pha.pa.us
In reply to: Thomas Lockhart (#2)
Re: pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Modified Files:

...

Some small polishing of Mark Hollomon's cleanup of DROP command: might
as well allow DROP multiple INDEX, RULE, TYPE as well. Add missing
CommandCounterIncrement to DROP loop, which could cause trouble otherwise
with multiple DROP of items affecting same catalog entries. Try to
bring a little consistency to various error messages using 'does not exist',
'nonexistent', etc --- I standardized on 'does not exist' since that's
what the vast majority of the existing uses seem to be.

Good idea(s). Thanks for cleaning up the error messages...

Speaking of error messages, one idea for 7.2 might be to prepended
numbers to the error messages. That way, people could look up a more
detailed description of the error and possible causes. Now, none of us
have the time to do that, but the new companies may, and they will need
those numbers to help with technical support anyway.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#3)
Re: pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Speaking of error messages, one idea for 7.2 might be to prepended
numbers to the error messages.

Isn't that long since on the TODO list? I know we've had long
discussions about a thoroughgoing revision of error reporting.

regards, tom lane

#5Bruce Momjian
pgman@candle.pha.pa.us
In reply to: Tom Lane (#4)
Re: pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Speaking of error messages, one idea for 7.2 might be to prepended
numbers to the error messages.

Isn't that long since on the TODO list? I know we've had long
discussions about a thoroughgoing revision of error reporting.

Yes. We have:

* Allow elog() to return error codes, not just messages
* Allow international error message support and add error codes

I just thought I would mention it is on my radar screen now.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
#6Peter Eisentraut
peter_e@gmx.net
In reply to: Bruce Momjian (#5)
Re: [COMMITTERS] pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Bruce Momjian writes:

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Speaking of error messages, one idea for 7.2 might be to prepended
numbers to the error messages.

Isn't that long since on the TODO list? I know we've had long
discussions about a thoroughgoing revision of error reporting.

Yes. We have:

* Allow elog() to return error codes, not just messages
* Allow international error message support and add error codes

I just thought I would mention it is on my radar screen now.

Yeah, it's on mine too. The only thing I'm still unsure about the
"international" part. Does anyone know of a gettext-ish thing that has an
acceptable license?

--
Peter Eisentraut peter_e@gmx.net http://yi.org/peter-e/

#7Bruce Momjian
pgman@candle.pha.pa.us
In reply to: Peter Eisentraut (#6)
Re: [COMMITTERS] pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Bruce Momjian writes:

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Speaking of error messages, one idea for 7.2 might be to prepended
numbers to the error messages.

Isn't that long since on the TODO list? I know we've had long
discussions about a thoroughgoing revision of error reporting.

Yes. We have:

* Allow elog() to return error codes, not just messages
* Allow international error message support and add error codes

I just thought I would mention it is on my radar screen now.

Yeah, it's on mine too. The only thing I'm still unsure about the
"international" part. Does anyone know of a gettext-ish thing that has an
acceptable license?

Yes, they must exist. I want a solution that doesn't make it difficult
for people add error messages. Having codes in the C files and error
messages in another file is quite a pain. My idea would enable us to
number the error messages, keep the English text for the message in the
same file next to the code, then allow international support by creating
a utility that can dump out all the codes with the Engligh text to allow
translators to make non-English versions. The translated file can then
be used by the backend to generate messages in other languagues using a
SET command.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
#8Vadim Mikheev
vmikheev@sectorbase.com
In reply to: Bruce Momjian (#3)
Re: [HACKERS] Re: pgsql/src/test/regress/expected (plpgsql.out inet.out foreign_key.out errors.out)

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Speaking of error messages, one idea for 7.2 might be to prepended
numbers to the error messages.

Isn't that long since on the TODO list? I know we've had long
discussions about a thoroughgoing revision of error reporting.

Yes, yes, yes! We need in numbers especially because of we
hopefully will have savepoints in 7.2 and so we would get powerful
error handling by *applications* not by *human* only.

Vadim