NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit45a6343

Barry Lind

committed

Patch from Ned Wolpert that fixes a bug that caused the cache of types not

to be used, causing extra sql statements to be executed. This was asignificant performance problem with the database meta data classes.The fix is a simple one liner.

1 parent4433eb1 commit45a6343Copy full SHA for 45a6343

File tree

1 file changed

-2

lines changed

src/interfaces/jdbc/org/postgresql
- Connection.java

1 file changed

-2

lines changed

`‎src/interfaces/jdbc/org/postgresql/Connection.java`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`importorg.postgresql.core.*;`
`12`	`12`
`13`	`13`	`/*`
`14`		`- * $Id: Connection.java,v 1.39 2001/11/25 23:26:56 barry Exp $`
	`14`	`+ * $Id: Connection.java,v 1.40 2001/12/11 04:44:23 barry Exp $`
`15`	`15`	`*`
`16`	`16`	`* This abstract class is used by org.postgresql.Driver to open either the JDBC1 or`
`17`	`17`	`* JDBC2 versions of the Connection class.`
`@@ -1103,7 +1103,7 @@ public boolean haveMinimumCompatibleVersion(String ver) throws SQLException`
`1103`	`1103`	`*/`
`1104`	`1104`	`publicintgetSQLType(intoid)throwsSQLException`
`1105`	`1105`	`{`
`1106`		`-IntegersqlType = (Integer)typeOidCache.get(newInteger(oid));`
	`1106`	`+IntegersqlType = (Integer)sqlTypeCache.get(newInteger(oid));`
`1107`	`1107`
`1108`	`1108`	`// it's not in the cache, so perform a query, and add the result to the cache`
`1109`	`1109`	`if (sqlType ==null)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit45a6343

File tree

1 file changed

1 file changed

`‎src/interfaces/jdbc/org/postgresql/Connection.java`

0 commit comments