Re: [HACKERS] Minor bug: inconsistent handling of overlength names - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [HACKERS] Minor bug: inconsistent handling of overlength names
Date
Msg-id 199808290236.WAA26663@candle.pha.pa.us
Whole thread Raw
In response to Minor bug: inconsistent handling of overlength names  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> DROP INDEX fails on overlength table names:
>
> tgl=> CREATE UNIQUE INDEX MarketOrderHistory_sequenceNo_Index
> tgl-> ON MarketOrderHistory USING btree (sequenceNo);
> CREATE
> tgl=> DROP INDEX MarketOrderHistory_sequenceNo_Index;
> ERROR:  pg_ownercheck: class "marketorderhistory_sequenceno_index" not found
> tgl=> DROP INDEX MarketOrderHistory_sequenceNo_I;
> DROP
>
> Evidently DROP INDEX is using a second-rate way of reducing the given
> name to canonical form for comparisons.
>
> Some further experimentation shows that CREATE TABLE won't let you
> create a relation name >= 32 characters in the first place.  So there's
> some inconsistency about what's done with overlength names.
>
> It seems to me that we ought to have consistent treatment of long names,
> and the treatment I like is the one that CREATE INDEX is using:
> silently truncate the given name to what we can handle, and accept
> it as long as the truncated form is unique.  This is the time-honored
> way of handling overlength names in compilers, and it works well.

OK. I have modified scan.l so it now truncates identifiers over
NAMEDATALEN, so this should fix it.

--
Bruce Momjian                          |  830 Blythe Avenue
maillist@candle.pha.pa.us              |  Drexel Hill, Pennsylvania 19026
  +  If your life is a hard drive,     |  (610) 353-9879(w)
  +  Christ can be your backup.        |  (610) 853-3000(h)

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] Segmentation fault with lo_export
Next
From: Bruce Momjian
Date:
Subject: Re: AW: [HACKERS] sequence creation