[jira] [Commented] (ARTEMIS-1922) Configuration reload does not work with include files

["Justin Bertram (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARTEMIS-1922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16508235#comment-16508235 ] 

Justin Bertram commented on ARTEMIS-1922:
-----------------------------------------

These two features can work together, but an additional step is required to trigger the reload - i.e. updating the timestamp on {{broker.xml}}.  I'll send a PR to update the documentation to clarify this.

To be clear, updating the timestamp on {{broker.xml}} is required because all the {{xinclude}} functionality is provided by the underlying XML parser since {{xinclude}} is a generic mechanism for merging XML documents and there was no desire to re-implement this in Artemis.  As far as the broker is concerned when it reads the XML elements from {{broker.xml}} it is just a single document because the parser has already done all the merging behind the scenes.  Since the broker doesn't know what was included from external files and what wasn't it has no way to monitor those files for updates (as it does with {{broker.xml}}).

> Configuration reload does not work with include files
> -----------------------------------------------------
>
>                 Key: ARTEMIS-1922
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1922
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>            Reporter: Lionel Cons
>            Priority: Major
>
> Artemis has two very useful features: configuration reload and using include files in {{broker.xml}} (aka XML XInclude).
> Unfortunately, these two do not work together as a change in a file included in {{broker.xml}} is not detected.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)