db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Knut Anders Hatlen (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (DERBY-4815) Override mechanism for modules.properties works backwards
Date Wed, 27 Oct 2010 07:42:19 GMT

     [ https://issues.apache.org/jira/browse/DERBY-4815?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Knut Anders Hatlen resolved DERBY-4815.
---------------------------------------

          Resolution: Fixed
       Fix Version/s: 10.7.1.0
    Issue & fix info:   (was: [Patch Available])

Thanks for looking at the patch, Dag.
Committed revision 1027847.

> Override mechanism for modules.properties works backwards
> ---------------------------------------------------------
>
>                 Key: DERBY-4815
>                 URL: https://issues.apache.org/jira/browse/DERBY-4815
>             Project: Derby
>          Issue Type: Bug
>          Components: Services
>    Affects Versions: 10.6.1.0
>            Reporter: Knut Anders Hatlen
>            Assignee: Knut Anders Hatlen
>             Fix For: 10.7.1.0
>
>         Attachments: d4815.diff
>
>
> It is possible to override the properties in org/apache/derby/modules.properties by putting
your own version of it somewhere on the classpath. BaseMonitor.getDefaultModuleProperties()
apparently intends to use values from the first modules.properties file that mentions a property:
>                             String key = (String) newKeys.nextElement();
>                             if( moduleList.contains( key))
>                                 // RESOLVE how do we localize messages before we have
finished initialization?
>                                 report( "Ignored duplicate property " + key + " in "
+ modulesPropertiesURL.toString());
>                             else
>                                 moduleList.setProperty( key, otherList.getProperty( key));
> However, moduleList.contains(key) doesn't look for a key in moduleList, it looks for
a property value. This code should have used containsKey() instead.
> Beacuse of this, the last modules.properties on the classpath will take precedence over
the ones earlier on the classpath. So if you for example have two different versions of derby.jar
in the classpath, the engine will use the classes from the first jar and modules.properties
from the last jar.

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