[jboss-jira] [JBoss JIRA] Updated: (JBRULES-3155) Bug with default properties in KnowledgeStoreServiceImpl

Bug with default properties in KnowledgeStoreServiceImpl -------------------------------------------------------- Key: JBRULES-3155 URL: https://issues.jboss.org/browse/JBRULES-3155 Project: Drools Issue Type: Bug Security Level: Public(Everyone can see) Components: drools-core Affects Versions: 5.2.0.Final Reporter: Pavel Sknar Assignee: Mark Proctor Priority: Minor Method setDefaultImplementations() set default properties in configProps (class KnowledgeStoreServiceImpl). mergeConfig() execute after specific properties loaded to configuration. This overlaps specific properties. [17:08] <pavlz> hi [17:09] <pavlz> I have bug for you [17:09] <Rikkola> :( [17:10] <pavlz> Problem with KnowledgeStoreServiceImpl [17:10] <pavlz> in org.drools.persistence.jpa [17:12] <pavlz> default properties merged first in config (in ChainedProperties: this.props.add( 0, properties )) [17:14] <pavlz> and this overlaps specific properties [17:15] <pavlz> drools.commandService, drools.timerService and else [17:15] <pavlz> I have this bug in 5.2.0.Final [17:23] <conan> pavlz: do you have a fix? [17:25] <pavlz> No, I see this now [17:27] <pavlz> but I think: need to create in ChainedProperties new method "setDefaultProperties" [17:29] <pavlz> and in SessionConfiguration too, to use this method in KnowledgeStoreServiceImpl.mergeConfig() [17:30] <pavlz> then default properties loaded later [17:31] <Rikkola> pavlz: best to make a jira about it [17:31] <Rikkola> https://issues.jboss.org/browse/JBRULES [17:31] <pavlz> ok