[DERBY-3525] Remove unneeded code to get JDBC level in BrokeredConnection and BrokeredStatement classes

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 10.5.1.1
• Fix Version/s: 10.3.3.0, 10.4.1.3, 10.5.1.1
• Component/s: JDBC
• Assignee: Katherine Marsden
• Reporter: Knut Anders Hatlen
• Created: 12/Mar/08 11:11
• Updated: 04/May/09 18:22
• Resolved: 19/Mar/08 17:14

Description

BrokeredConnection has a method called getJDBCLevel() whose only purpose is to provide a value that can be stored in BrokeredStatement.jdbcLevel. This field is only used once, in BrokeredStatement.createDuplicateStatement():

if (jdbcLevel == 2)
newStatement = conn.createStatement(resultSetType, resultSetConcurrency);
else
newStatement = conn.createStatement(resultSetType, resultSetConcurrency,
resultSetHoldability);

Since getJDBCLevel() only returns 2 if Java version 1.3 is used, and Derby doesn't support Java 1.3 any more, BrokeredConnection.getJDBCLevel() and BrokeredStatement.jdbcLevel could be removed.

Issue Links

• is related to: DERBY-3513 NullPointerException in newBrokeredStatement in app server environment