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

Kathey Marsden commented on DERBY-3839:
---------------------------------------

Thanks Tiago for the patch.  The test looks good, but we need to add the new 
test to store._Suite and remove the old test from derbynetmats.runall, 
storemore.runall and j9derbynetmats.runall.

It might be cleanest to change the result set to be updatable as Dag suggests 
to get consistent client/embedded behavior and add a comment as to why this was 
done.


> 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, 
> derby-3839.patch, ReproHoldCursorBug.java, ReproHoldCursorBug.java, 
> ReproHoldCursorBug.java
>
>


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

Reply via email to