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

[Toshiya Kobayashi (JIRA)]

    [ https://issues.jboss.org/browse/JBRULES-3155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709895#comment-12709895 ] 

Toshiya Kobayashi commented on JBRULES-3155:
--------------------------------------------

Sent 2 pull requests following Pavel's idea.
                
> 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

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira