From: | Rene Pijlman <rpijlman(at)wanadoo(dot)nl> |
---|---|
To: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Cc: | Greg Zoller <gzoller(at)hotmail(dot)com>, pgsql-patches(at)postgresql(dot)org, PostgreSQL jdbc list <pgsql-jdbc(at)postgresql(dot)org> |
Subject: | Re: Re: [PATCHES] JDBC2 Array Patch (New feature) |
Date: | 2001-08-08 20:50:45 |
Message-ID: | fu83nto5a0u6gcc4jd0nkt0j3ih80n8iin@4ax.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc pgsql-patches |
On Wed, 8 Aug 2001 11:19:32 -0400 (EDT), Greg Zoller wrote:
>> Specifically you can now use the ResultSet.getArray() method
>> to return an java.sql.Array object. Its a really convenient
>> way to load up a Java array with one columns' worth of results.
I don't think this patch is a correct implementation of the JDBC
spec. I've mailed this to Greg:
R>Isn't ResultSet.getArray() supposed to fetch *one* value
R>(the value of one column in one row) and interpret that as
R>a structured value of an SQL array field
R>(http://www.postgresql.org/users-lounge/docs/7.1/postgres/arrays.html)?
R>It looks like your code is looping through the ResultSet
R>and gathering data from multiple rows.
And received this reply:
G>You know what... I may have misinterpreted the JDBC spec!
G>My original understanding was that getArray was supposed to
G>iterate through a ResultSet and load up a Java array with
G>the contents of a given field (i.e. column). But now as
G>I re-read the specification it does actually mean the
G>interpretation you gave (load a single field of type array
G>into a Java array object).
G>
G>So it seems that my implementation of Array is based on a
G>misunderstanding. :-(
G>
G>Massive bummer. Please disregard my patch.
Greg, I do hope you continue your efforts! Not bad for a first
patch anyway, mine was only 2 lines. We still need Array support
badly... (hint, hint) :-)
Regards,
René Pijlman
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2001-08-08 20:50:48 | Re: [PATCHES] JDBC2 Array Patch (New feature) |
Previous Message | Bruce Momjian | 2001-08-08 20:42:07 | Re: [PATCHES] JDBC Statement cleanup patch |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2001-08-08 20:50:48 | Re: [PATCHES] JDBC2 Array Patch (New feature) |
Previous Message | Bruce Momjian | 2001-08-08 20:42:07 | Re: [PATCHES] JDBC Statement cleanup patch |