Thread

1. varchar-array.patch applied

   Tom Lane <tgl@sss.pgh.pa.us> — 1999-05-04T13:51:45Z

   > 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
   
   

2. Re: [HACKERS] varchar-array.patch applied

   Thomas Lockhart <lockhart@alumni.caltech.edu> — 1999-05-04T14:40:40Z

   > > 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
   
   

3. Re: [HACKERS] varchar-array.patch applied

   Bruce Momjian <maillist@candle.pha.pa.us> — 1999-05-04T16:55:03Z

   > > 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
   
   

4. Re: [HACKERS] varchar-array.patch appliedu

   Massimo Dal Zotto <dz@cs.unitn.it> — 1999-05-04T19:27:02Z

   > 
   > > > 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  |
   +----------------------------------------------------------------------+