Re: type cache info fix

From: Valentine Gogichashvili <valgog(at)gmail(dot)com>
To: Gary Baker <gary(dot)baker(at)gmail(dot)com>
Cc: pgsql-jdbc(at)postgresql(dot)org
Subject: Re: type cache info fix
Date: 2013-04-12 01:40:01
Message-ID: CAP93muXTS4UMZv0w_cp6Se0ATqQwMp_waTcCPE7LBa9QC2rfWw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

Hello Gary,

the problem with this change is that it is not backwards compatible. Though
the question is, why somebody would like to keep a not deterministic
behavior of the current code.

And if you are fixing this, probably it would make sense to fix
getSQLType() as well.

Also it can fill your server logs with errors, if the type does not exist,
regtype cast will throw an ERROR

I have prepared a little bit different change:
https://github.com/pgjdbc/pgjdbc/pull/52

Cheers,

-- Valentine

ვალენტინ გოგიჩაშვილი
Valentine Gogichashvili

On Fri, Mar 29, 2013 at 6:08 PM, Gary Baker <gary(dot)baker(at)gmail(dot)com> wrote:

> I found that type cache info wasn't honoring search path. (I had a
> UDT type that shared a name with a table).
>
> This fixed it.
>
>
> diff --git a/org/postgresql/jdbc2/TypeInfoCache.java
> b/org/postgresql/jdbc2/TypeInfoCache.java
> index 2e79758..2d26fd0 100644
> --- a/org/postgresql/jdbc2/TypeInfoCache.java
> +++ b/org/postgresql/jdbc2/TypeInfoCache.java
> @@ -234,7 +234,7 @@ public class TypeInfoCache implements TypeInfo {
> if (_getOidStatement == null) {
> String sql;
> if (_conn.haveMinimumServerVersion("7.3")) {
> - sql = "SELECT oid FROM pg_catalog.pg_type WHERE typname =
> ?";
> + sql = "SELECT ?::regtype::oid";
> } else {
> sql = "SELECT oid FROM pg_type WHERE typname = ?";
> }
>
>
> --
> Sent via pgsql-jdbc mailing list (pgsql-jdbc(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>

In response to

Responses

Browse pgsql-jdbc by date

  From Date Subject
Next Message Kevin Grittner 2013-04-12 02:32:54 Re: Improvements for statement cancellation
Previous Message John R Pierce 2013-04-11 01:55:49 Re: Improvements for statement cancellation