db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kathey Marsden (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DERBY-4509) Convert autoincrement.sql to JUnit
Date Thu, 04 Mar 2010 21:45:27 GMT

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

Kathey Marsden commented on DERBY-4509:
---------------------------------------

I spoke with Jayaram on the phone and IRC about the Autoincrementrevised.txt patch.
The main thing was that he needs to review the new test and compare it to  autoincrement.out
and make sure everything is included. I put suggested he put it in the same general order
 and copy the comments exactly so we don't miss anything and it is easy to check.   Besides
that, I had the following  details:
- suite.addTest(OrderByAndOffsetFetchInSubqueries.suite()); seems to be missing now from lang/_Suite.java
- autoincrement.sql is still in derbylang.runall and needs to be removed.
- test name in AutoIncrementTest in licence header should be changed from SimpleTest
- use local variables for rs and expectedRows in each fixture instead of fields.  Fields tend
to leak memory in Junit tests.
- make sure lock_table queries are being executed as they were in autoincrement.sql. The view
creation is there but I didn't see the queries to check for locking behavior.


> Convert autoincrement.sql to JUnit
> ----------------------------------
>
>                 Key: DERBY-4509
>                 URL: https://issues.apache.org/jira/browse/DERBY-4509
>             Project: Derby
>          Issue Type: Task
>          Components: Test
>         Environment: Apache Derby
>            Reporter: Jayaram Subramanian
>            Assignee: Jayaram Subramanian
>            Priority: Minor
>         Attachments: AutoIncrementDO-NOT_COMMIT_diff.txt, AutoIncrementDO_NOT_COMMIT_diff.txt,
AutoIncrementDO_NOT_COMMIT_diff.txt, AutoIncrementrevised.txt, jira4509.txt, jira4509feb4.txt,
jira4509feb4stat.txt, jira4509stat.txt, statrevised.txt
>
>
> The task is to convert existing autoincrement.sql to JUnit

-- 
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