From: <ji...@co...> - 2004-07-31 21:42:39
|
Message: A new issue has been created in JIRA. --------------------------------------------------------------------- View the issue: http://jira.codehaus.org/browse/GEOT-224 Here is an overview of the issue: --------------------------------------------------------------------- Key: GEOT-224 Summary: primary key of type DECIMAL fails Type: Bug Status: Unassigned Priority: Major Original Estimate: 15 minutes Time Spent: Unknown Remaining: 15 minutes Project: GeoTools Components: oraclespatial Fix Fors: 2.0-RC1 2.1.M0 Versions: 2.0.RC0 Assignee: Reporter: James Macgill Created: Sat, 31 Jul 2004 5:42 PM Updated: Sat, 31 Jul 2004 5:42 PM Description: cannot load Oracle tables which have a single column primary key of type DECIMAL. The same problem would happen with a DATE type. The multi column fid mapper handles DECIMALS / arbitrary types fine, so can we please change FIDMapper buildSingleColumnFidMapper to be consistent. Instead of returning null, it just needs to return a BasicFIDMapper : eg from this private FIDMapper buildSingleColumnFidMapper(String schema, String tableName, Connection connection, ColumnInfo ci) { if (ci.autoIncrement) { return new AutoIncrementFIDMapper(ci.colName, ci.dataType); } else if (isIntegralType(ci.dataType)) { return new MaxIncFIDMapper(tableName, ci.colName, ci.dataType); } else if (isTextType(ci.dataType)) { return new BasicFIDMapper(ci.colName, ci.size); } else { return null; } } to this private FIDMapper buildSingleColumnFidMapper(String schema, String tableName, Connection connection, ColumnInfo ci) { if (ci.autoIncrement) { return new AutoIncrementFIDMapper(ci.colName, ci.dataType); } else if (isIntegralType(ci.dataType)) { return new MaxIncFIDMapper(tableName, ci.colName, ci.dataType); } else { return new BasicFIDMapper(ci.colName, ci.size); } --------------------------------------------------------------------- JIRA INFORMATION: This message is automatically generated by JIRA. If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa If you want more information on JIRA, or have a bug to report see: http://www.atlassian.com/software/jira |