You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Daniel John Debrunner (JIRA)" <ji...@apache.org> on 2007/01/15 19:39:27 UTC
[jira] Created: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
-------------------------------------------------------------------------------------------------
Key: DERBY-2243
URL: https://issues.apache.org/jira/browse/DERBY-2243
Project: Derby
Issue Type: Bug
Components: JDBC
Reporter: Daniel John Debrunner
Priority: Trivial
Since this method is returning the support level of SQL it must be consistent across the drivers.
Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Daniel John Debrunner (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466130 ]
Daniel John Debrunner commented on DERBY-2243:
----------------------------------------------
Patch looks fine though I personally don't like the habit of just commenting out the old code. If that practice is followed then after a while the code becomes unreadable. SVN history provides the ability to see old code.
Also I think you need to modifiy jdbcapi.DatabaseMetaDataTest, since it is checking the output of these methods.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466212 ]
Saurabh Vyas commented on DERBY-2243:
-------------------------------------
Thanks Dan for your comments and guiding me for best practices (i.e. removing the old code rather than just commenting it)
Incorporated the changes & modified jdbcapi.DatabaseMetaDataTest to reflect the changes (i.e. TRUE as return value for these methods).
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat, Derby-2243_v2.diff, Derby-2243_v2.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Saurabh Vyas updated DERBY-2243:
--------------------------------
Derby Info: [Patch Available]
Fix Version/s: 10.3.0.0
Modifying the return type for methods like
- supportsANSI92EntryLevelSQL()
- supportsExpressionsInOrderBy()
in EmbedDatabaseMetaData.java to return TRUE as Derby does support these features
Thus now the behavior is consistent across drivers.
This patch address DERBY-2244 (https://issues.apache.org/jira/browse/DERBY-2244) also.
Accordingly modified the master output files also as earlier these methods were expected to return FALSE but now they will return TRUE value.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12470904 ]
Saurabh Vyas commented on DERBY-2243:
-------------------------------------
Thanks alot for modifying the assert condition in DatabaseMetaDataTest.java (Just looked at the comments in the test code for DERBY-2243 & DERBY-2244.)
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat, Derby-2243_v2.diff, Derby-2243_v2.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Saurabh Vyas updated DERBY-2243:
--------------------------------
Attachment: Derby-2243_v1.stat
Derby-2243_v1.diff
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Assigned: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Saurabh Vyas reassigned DERBY-2243:
-----------------------------------
Assignee: Saurabh Vyas
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466112 ]
Saurabh Vyas commented on DERBY-2243:
-------------------------------------
I looked into the code and found that DatabaseMetaData.supportsANSI92EntryLevelSQL() is implements in different way in Embedded and Network-Client mode.
In embedded mode EmbedDatabaseMetaData implements DatabaseMetaData & it returns false for various functions including supportsANSI92EntryLevelSQL() which is incorrect. Whereas for network mode, NetDatabaseMetaData implements DatabaseMetaData & it returns true for supportsANSI92EntryLevelSQL().
I crossed check it with MySql code & found that it also return true (which is expected ) for supportsANSI92EntryLevelSQL().
Same is the case with DERBY-2244 (https://issues.apache.org/jira/browse/DERBY-2244) & it returns false for supportsExpressionsInOrderBy() which is incorrect as derby does supports Order By Clause.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12468256 ]
Saurabh Vyas commented on DERBY-2243:
-------------------------------------
Thanks a lot Dan. I had files for ICLA now.
Apart I had verified for this issue & Derby-2244, and now these methods returned the correct values.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat, Derby-2243_v2.diff, Derby-2243_v2.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466128 ]
Saurabh Vyas commented on DERBY-2243:
-------------------------------------
My apologies on this and thanks Dan for correcting me. Well what I was trying to see that how other Databases handled this scenario, but I was not sure of this licensing issue. Thanks again.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Daniel John Debrunner (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466121 ]
Daniel John Debrunner commented on DERBY-2243:
----------------------------------------------
MySQL's return value should have no impact on Derby, it's if Derby supports these features only.
Also, please be careful looking at MySQL code, bringing MySQL GPL code into Derby's codebase would not be acceptable.
Obviously in this case a return true/false is not an issue, but demonstrated access to a piece of work is enough for copyright infringements.
E.g. if someone says I looked at MySQL code to see how X was done and then implemented the same functionality in Derby in a similar manner that could be enough to pollute the Derby code base with GPL.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Daniel John Debrunner (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel John Debrunner updated DERBY-2243:
-----------------------------------------
Priority: Minor (was: Trivial)
Affects Version/s: 10.3.0.0
10.2.1.6
10.2.2.0
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Priority: Minor
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Saurabh Vyas updated DERBY-2243:
--------------------------------
Attachment: Derby-2243_v2.stat
Derby-2243_v2.diff
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat, Derby-2243_v2.diff, Derby-2243_v2.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Resolved: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Saurabh Vyas (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Saurabh Vyas resolved DERBY-2243.
---------------------------------
Resolution: Fixed
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat, Derby-2243_v2.diff, Derby-2243_v2.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-2243)
DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded,
true for client driver
Posted by "Daniel John Debrunner (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-2243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel John Debrunner updated DERBY-2243:
-----------------------------------------
Derby Info: (was: [Patch Available])
Patch Derby-2243_v2.diff committed revision 498732 - Thanks Saurabh Vyas.
BTW - I see that you do not have an ICLA on file at Apache, if you are intending to work more on Derby I would encourage you to submit an ICLA ahead of time rather than waiting for a significant contribution.
> DatabaseMetaData.supportsANSI92EntryLevelSQL() returns false for embedded, true for client driver
> -------------------------------------------------------------------------------------------------
>
> Key: DERBY-2243
> URL: https://issues.apache.org/jira/browse/DERBY-2243
> Project: Derby
> Issue Type: Bug
> Components: JDBC
> Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.0.0
> Reporter: Daniel John Debrunner
> Assigned To: Saurabh Vyas
> Priority: Minor
> Fix For: 10.3.0.0
>
> Attachments: Derby-2243_v1.diff, Derby-2243_v1.stat, Derby-2243_v2.diff, Derby-2243_v2.stat
>
>
> Since this method is returning the support level of SQL it must be consistent across the drivers.
> Probably need some investigation as to which is correct, most likely 'true'.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira