Hi Lance, I looked at the review package and it looks good. Good comments in the EmbedDatabaseMetaData.java. Just wondered if you ran the existing tests to make sure nothing got affected by the change.
Mamta On 5/4/05, Lance J. Andersen <[EMAIL PROTECTED]> wrote: > Oh, I forgot to mention that i validated that the dbMetaDataJdbc30 in jdk14 > passed. > > > > Lance Andersen wrote: > > Attached is the patch. > > Regards > Lance > > Mamta Satoor wrote: > Hi Lance, > > Thanks for taking it. Yes, it is for Embedded Driver. > > Mamta > > On 4/29/05, Lance J. Andersen <[EMAIL PROTECTED]> wrote: > > I take it this is with the Embedded Driver as it looks like the code for the > new Client Driver is returning false. > > I can take this if no one else wants it. > > > Mamta Satoor wrote: > Hi, > > > I think this is a very easy bug to fix. So, if someone is looking > > for > > an opportunity to start with a simple bug, this will be a good > > one. > > > Mamta > > On 4/26/05, Satheesh Bandaram (JIRA) <[email protected]> > > wrote: > > > > DatabaseMetaData.supportsGetGeneratedKeys needs to return > FALSE, since Derby only has limited > support. > > ------------------------------------------------------------------------------------------------------ > > > Key: DERBY-242 > > URL: > > http://issues.apache.org/jira/browse/DERBY-242 > > Project: > > Derby > > Type: Bug > Components: JDBC > Versions: 10.0.2.0 > Environment: ALL > > Reporter: Satheesh Bandaram > > Assigned to: Satheesh Bandaram > Fix For: > > 10.1.0.0 > > > Derby only has partial support for retrieving generated > > values. > > Because of this, it is desirable to > > make > > DatabaseMetaData.supportsGetGeneratedKeys return > > FALSE. One > > of the restrictions include Jira entry Derby-201. Till this > > is > > fixed, we should change derby to return FALSE for this. > > This would also > > match Derby client behavior. > > > -- > 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 > > > > > > > > > ________________________________ > Index: > java/engine/org/apache/derby/impl/jdbc/EmbedDatabaseMetaData.java =================================================================== --- > java/engine/org/apache/derby/impl/jdbc/EmbedDatabaseMetaData.java > (revision 168122) +++ > java/engine/org/apache/derby/impl/jdbc/EmbedDatabaseMetaData.java > (working copy) @@ -2880,7 +2880,13 @@ */ public boolean > supportsGetGeneratedKeys() { - return true; + /* + * Currently reverting > the returned value to false until there + * is more support for > autogenerated keys in Derby. + * (such as support for specifying the > returned columns for + * the autogenerated key) + */ + return false; } > /** Index: > java/testing/org/apache/derbyTesting/functionTests/master/dbMetaDataJdbc30.out =================================================================== --- > java/testing/org/apache/derbyTesting/functionTests/master/dbMetaDataJdbc30.out > (revision 168122) +++ > java/testing/org/apache/derbyTesting/functionTests/master/dbMetaDataJdbc30.out > (working copy) @@ -4,7 +4,7 @@ supportsSavepoints() : true > supportsNamedParameters() : false supportsMultipleOpenResults() : > true -supportsGetGeneratedKeys() : true +supportsGetGeneratedKeys() : false > supportsResultSetHoldability(HOLD_CURSORS_OVER_COMMIT) : > true supportsResultSetHoldability(CLOSE_CURSORS_AT_COMMIT) > : true getJDBCMajorVersion() : 3 >
