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-4217) Make the default port for the suites.All run configurable with a system property.
Date Tue, 19 May 2009 22:29:45 GMT

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

Kathey Marsden commented on DERBY-4217:

>From a functional perspective, when tests are run,  we want the value set by derby.tests.port
to override the 1527 setting.  Ultimately I think we will need three system properties derby.tests.port,
derby.tests.alternativePort (currently used by NSinSameJVMTest) and derby.tests.replicationPort.

The question becomes "Where do  we want to process these system properties?", in TestConfiguration
itself or in the code that calls the TestConfiguration constructor with the port argument?
  For example this is some code from ReplicationTestRun which ultimately calls the TestConfiguration
constructor with the port. It looks like someone already tried to partially implement this
system property business before, so things look a bit tangled up.

 String masterHostName = System.getProperty("test.serverHost", "localhost");
        int masterPortNo = Integer.parseInt(System.getProperty("test.serverPort", "1527"));
        suite.addTest(StandardTests.simpleTest(masterHostName, masterPortNo));
        System.out.println("*** Done suite.addTest(StandardTests.simpleTest())");

Let me study this a bit and get back to you tomorrow with a recommended approach.  Meanwhile
if anyone else wants to jump in, suggestions are welcome.

> Make the default port for the suites.All run configurable with a system property.
> ---------------------------------------------------------------------------------
>                 Key: DERBY-4217
>                 URL: https://issues.apache.org/jira/browse/DERBY-4217
>             Project: Derby
>          Issue Type: Sub-task
>    Affects Versions:
>            Reporter: Tiago R. Espinha
>            Assignee: Tiago R. Espinha
>         Attachments: DERBY-4217.patch, DERBY-4217.patch, DERBY-4217.patch, DERBY-4217.patch,
DERBY-4217.stat, DERBY-4217.stat, ReproNetworkServerControl.java
> The goal is to make the port used for suites.All configurable through a system property
passed on to the JVM.

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

View raw message