Ugh, sorry. My email didn't make sense, as I read it. I should know
better than trying to communicate late on a Friday....
The problem is typeOidCache is used in getSQLType(int) rather than
sqlTypeCache. Diff is here.
[wolpert(at)osti jdbc]$ cvs diff -e org/postgresql/Connection.java
retrieving revision 1.39
diff -e -r1.39 Connection.java
Integer sqlType = (Integer)sqlTypeCache.get(new
On Fri, 2001-12-07 at 15:39, Ned Wolpert wrote:
> This was right on the tip of my sight, but I missed it in my last email
> Bug in the method getSQLType(int oid). It looks for an integer in
> Integer sqlType = (Integer)typeOidCache.get(iOid);
> But when none found, it does a query to the database to get the value.
> But it never puts it into the hashtable/cache. The method does the
> sqlTypeCache.put(iOid, sqlType);
> pgTypeCache.put(iOid, pgType);
> One of two things should happend.
> 1) change line (1118) from
> 2) add in the line
> in line 1118
> My question is, that sqlTypeCache is only used here, but typeOidCache is
> used in several places. Should we remove sqlTypeCache and only use
> typeOidCache or are both needed?
> Ned Wolpert <ned(dot)wolpert(at)knowledgenet(dot)com>
> D08C2F45: 28E7 56CB 58AC C622 5A51 3C42 8B2B 2739 D08C 2F45
Ned Wolpert <ned(dot)wolpert(at)knowledgenet(dot)com>
D08C2F45: 28E7 56CB 58AC C622 5A51 3C42 8B2B 2739 D08C 2F45
In response to
pgsql-jdbc by date
|Next:||From: Thomas O'Dowd||Date: 2001-12-08 05:40:51|
|Subject: patch against cvs for getTimestamp() problem.|
|Previous:||From: Ned Wolpert||Date: 2001-12-07 22:39:26|
|Subject: Bug in org.postgresql.Connection|