[ http://issues.apache.org/jira/browse/DERBY-1527?page=all ]
Knut Anders Hatlen closed DERBY-1527.
-------------------------------------
Fix Version/s: 10.2.0.0
Resolution: Fixed
Derby Info: (was: [Patch Available])
Thanks for the review, Kristian. I think jdbcapi/parameterMapping.java tests
data types for setXXX and getXXX methods (but not updateXXX).
Committed revision 423912.
> Factor out type checks in EmbedResultSet and EmbedPreparedStatement
> -------------------------------------------------------------------
>
> Key: DERBY-1527
> URL: http://issues.apache.org/jira/browse/DERBY-1527
> Project: Derby
> Issue Type: Improvement
> Components: JDBC
> Affects Versions: 10.2.0.0
> Reporter: Knut Anders Hatlen
> Assigned To: Knut Anders Hatlen
> Priority: Minor
> Fix For: 10.2.0.0
>
> Attachments: derby-1527-v1.diff, derby-1527-v1.stat
>
>
> EmbedPreparedStatement and EmbedResultSet have many occurrences of
> switch (...) {
> case Types.CHAR:
> case Types.VARCHAR:
> case Types.LONGVARCHAR:
> case Types.CLOB:
> break;
> default:
> throw dataTypeConversion(...);
> }
> and
> switch (...) {
> case Types.BINARY:
> case Types.VARBINARY:
> case Types.LONGVARBINARY:
> case Types.BLOB:
> break;
> default:
> throw dataTypeConversion(...);
> }
> Instead of having many copies of this code, it would be good to factor
> it out into methods that can be shared. For instance, one could add
> these methods to DataTypeDescriptor:
> public static boolean isAsciiStreamCompatible(int jdbcType)
> public static boolean isBinaryStreamCompatible(int jdbcType)
> public static boolean isCharacterStreamCompatible(int jdbcType)
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
