varchar-array.patch applied

Started by Tom Laneover 26 years ago4 messages
#1Tom Lane
tgl@sss.pgh.pa.us

varchar-array.patch this patch adds support for arrays of bpchar() and
varchar(), which where always missing from postgres.

Far be it from me to carp ... but I thought adding new features during
beta phase was frowned upon.

regards, tom lane

#2Thomas Lockhart
lockhart@alumni.caltech.edu
In reply to: Tom Lane (#1)
Re: [HACKERS] varchar-array.patch applied

this patch adds support for arrays of bpchar() and
varchar(), which where always missing from postgres.

... but I thought adding new features during
beta phase was frowned upon.

This one is probably in a grey area, since the feature was explicitly
disabled back in the *very* early days by Jolly, but with a comment
from her that she wasn't sure it was necessary.

If we have a hint of trouble, we can back it out. It sure would be
nice to have Massimo also contribute a patch to the relevant
regression tests ;)

- Tom

--
Thomas Lockhart lockhart@alumni.caltech.edu
South Pasadena, California

#3Bruce Momjian
maillist@candle.pha.pa.us
In reply to: Tom Lane (#1)
Re: [HACKERS] varchar-array.patch applied

varchar-array.patch this patch adds support for arrays of bpchar() and
varchar(), which where always missing from postgres.

Far be it from me to carp ... but I thought adding new features during
beta phase was frowned upon.

Yes, but he sent it in before the freeze, and I was too busy to apply
it. Seems only fair.

-- 
  Bruce Momjian                        |  http://www.op.net/~candle
  maillist@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
#4Massimo Dal Zotto
dz@cs.unitn.it
In reply to: Thomas Lockhart (#2)
Re: [HACKERS] varchar-array.patch appliedu

this patch adds support for arrays of bpchar() and
varchar(), which where always missing from postgres.

... but I thought adding new features during
beta phase was frowned upon.

This one is probably in a grey area, since the feature was explicitly
disabled back in the *very* early days by Jolly, but with a comment
from her that she wasn't sure it was necessary.

If we have a hint of trouble, we can back it out. It sure would be
nice to have Massimo also contribute a patch to the relevant
regression tests ;)

- Tom

--
Thomas Lockhart lockhart@alumni.caltech.edu
South Pasadena, California

The patch was first submitted before the beta phase but never applied.

It adds features which replace other features which were dropped in 6.3,
specifically char4 and friends with related array types. I had many tables
with _char8 and _char16 fields which I couldn't use anymore.

The new code is never executed unless you try to to define arrays of
varchar or bpchar, so it shoud be safe for existing applications.

Anyway I will try to write some regression tests.

--
Massimo Dal Zotto

+----------------------------------------------------------------------+
|  Massimo Dal Zotto               email: dz@cs.unitn.it               |
|  Via Marconi, 141                phone: ++39-0461534251              |
|  38057 Pergine Valsugana (TN)      www: http://www.cs.unitn.it/~dz/  |
|  Italy                             pgp: finger dz@tango.cs.unitn.it  |
+----------------------------------------------------------------------+