db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tiago R. Espinha (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DERBY-3839) Convert "org.apache.derbyTesting.functionTests.tests.store.holdCursorJDBC30.sql" to junit.
Date Thu, 23 Apr 2009 18:39:31 GMT

    [ https://issues.apache.org/jira/browse/DERBY-3839?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12702051#action_12702051
] 

Tiago R. Espinha commented on DERBY-3839:
-----------------------------------------

After talking this over on IRC, Dag suggested something that fixed this issue.

This test is using the following parameters on the createStatement:
ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_UPDATABLE and ResultSet.HOLD_CURSORS_OVER_COMMIT

After going through the ResultSet code, I found out that the checkForUpdatableResultSet()
method (called upon the deleteRow() )  overrides any of the checks that it would do, unless
the ResultSet is set as CONCUR_READ_ONLY. So, basically put, switching this parameter will
make the repro pass.

I am still working on the actual test and at this point I decided that it would be a better
option to create individual tables for each test.
(As suggested by Kathey here: http://tinyurl.com/dxozhb )

Finally, I am also opening a bug for the described behavior above.

> Convert "org.apache.derbyTesting.functionTests.tests.store.holdCursorJDBC30.sql" to junit.

> -------------------------------------------------------------------------------------------
>
>                 Key: DERBY-3839
>                 URL: https://issues.apache.org/jira/browse/DERBY-3839
>             Project: Derby
>          Issue Type: Test
>          Components: Test
>            Reporter: Junjie Peng
>            Assignee: Tiago R. Espinha
>         Attachments: derby-3839-1.patch, derby-3839-1.stat, derby-3839.patch, ReproHoldCursorBug.java
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message