[jboss-jira] [JBoss JIRA] (WFWIP-54) [JDBC HA] Artemis system properties override model configuration

[Jeff Mesnil (JIRA)]

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

Jeff Mesnil commented on WFWIP-54:
----------------------------------

Issue is addressed in https://github.com/wildfly/wildfly/pull/11355 in https://github.com/wildfly/wildfly/pull/11355/commits/feef6780137f85aee00b1889278be0abe04eb28f to remove the corresponding Artemis System properties

> [JDBC HA] Artemis system properties override model configuration
> ----------------------------------------------------------------
>
>                 Key: WFWIP-54
>                 URL: https://issues.jboss.org/browse/WFWIP-54
>             Project: WildFly WIP
>          Issue Type: Bug
>          Components: JMS
>            Reporter: Erich Duda
>            Assignee: Jeff Mesnil
>
> This issue is related to https://github.com/wildfly/wildfly/pull/11355.
> Based on the Analysis document \[1\] and because of backward compatibility, new JDBC HA configuration parameters can be configured in two ways. Using the system properties and using the management model. It was decided that values defined in management model has higher priority than values defined by system properties. However this rule is not applied.
> I found out that although integration code passes correct values to Artemis configuration, they are replaced by a method \[2\]. I think that integration code should unset the system properties so Artemis cannot read them.
> \[1\] https://github.com/wildfly/wildfly-proposals/pull/87/
> \[2\] https://github.com/rh-messaging/jboss-activemq-artemis/blob/1.5.5.jbossorg-012/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ActiveMQServerImpl.java#L507



--
This message was sent by Atlassian JIRA
(v7.5.0#75005)