Re: JDBC Array Support, Take III

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Greg Zoller <gzoller(at)hotmail(dot)com>
Cc: barry(at)xythos(dot)com, pgsql-patches(at)postgresql(dot)org
Subject: Re: JDBC Array Support, Take III
Date: 2001-08-17 14:45:03
Message-ID: 200108171445.f7HEj3d21705@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc pgsql-patches


Patch applied. I have attached a non-gzip version for review.

> Greetings again.
>
> Thanks for your feedback (and patience). Enclosed is my third
> attempt at a patch to 7.1.2 to support Array.
>
> [I think I've solved the mangled patch problem. Hotmail seems to
> try to format the text file, so gzipping it should solve this
> problem.]
>
> In this patch I've incorporated Barry's feedback. Specifically:
>
> 1) OIDs are no longer hard-coded into Array.java. In order to
> support this change I added a getOID(String) method to Field.java
> which receives a PostgreSQL field type and returns a value from
> java.sql.Types. I couldn't get away from using OIDs altogether
> because the JDBC spec for Array specifies that some methods return
> a ResultSet. This requires I construct Field objects,
> which means I need OIDs. At least this approach doesn't hard
> code these values. A Hashtable cache has been added to Field
> so that an SQL lookup isn't necessary (following the model already
> in Field.java).
>
> 2) Rewired the base formatting code in ResultSet.java to use 'to'
> methods, which are then exposed as static methods in ResultSet.
> These methods are used in Array to format the data without
> duplications in the code.
>
> 3) Artifact call to first() in ResultSet.getArray() removed.
>
> Patching as before: (gunzip first this time)
>
> >cd <PGSQL_SRC_PATH>/src/interfaces/jdbc/org/postgresql
> >patch -p1 < $PATH_TO_PATCH/patch
> >cd <PGSQL_SRC_PATH>
> >make
>
> Let me know if I'm getting closer to something useful.
>
> Take care.
> Greg
>
> _________________________________________________________________
> Get your FREE download of MSN Explorer at http://explorer.msn.com/intl.asp

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 2: you can get off all lists at once with the unregister command
> (send "unregister YourEmailAddressHere" to majordomo(at)postgresql(dot)org)

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

Attachment Content-Type Size
unknown_filename text/plain 18.8 KB

In response to

Browse pgsql-jdbc by date

  From Date Subject
Next Message Rene Pijlman 2001-08-17 15:16:37 Re: postgresql ID
Previous Message Bruce Momjian 2001-08-17 13:59:31 Re: [PATCHES] Re: JDBC pg_description update needed for CVS tip

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2001-08-17 14:46:21 Re: Patch for JDBC to update some comments
Previous Message Bruce Momjian 2001-08-17 13:59:31 Re: [PATCHES] Re: JDBC pg_description update needed for CVS tip