db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rick Hillegas (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DERBY-3921) Replication: Test replication of encrypted databases
Date Mon, 09 Mar 2009 15:40:50 GMT

    [ https://issues.apache.org/jira/browse/DERBY-3921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12680177#action_12680177

Rick Hillegas commented on DERBY-3921:

Thanks for the patch Ole. The replication suite ran cleanly for me after applying these changes.
I have a couple small comments on the patch:

Style: In a number of these tests, the setUp() and tearDown() overrides don't seem to do anything.
I'd be inclined to remove these methods.

Headers: I like the fact that some of the test cases have JIRA numbers embedded in their names.
This helps connect the test cases with the issues. It would be even more helpful if the test
cases had header comments which explained more about what was being tested.

Factoring: I see a lot of instances of code like this, which I'd recommend abstracting into
a shared method:

+           URL = DB_PROTOCOL
+                +"://"+masterServerHost
+                +":"+masterServerPort+"/"
+                +masterDatabasePath+FS+masterDbSubPath+FS+replicatedDb
+                +useEncryption(false);


> Replication: Test replication of encrypted databases
> ----------------------------------------------------
>                 Key: DERBY-3921
>                 URL: https://issues.apache.org/jira/browse/DERBY-3921
>             Project: Derby
>          Issue Type: Test
>          Components: Replication, Test
>    Affects Versions:
>            Reporter: Jørgen Løland
>            Assignee: Ole Solberg
>         Attachments: DERBY-3921_diff.txt, DERBY-3921_stat.txt
> It should be possible to replicate encrypted databases. Tests are needed to ensure that
this works.

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

View raw message