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

ASF subversion and git services commented on ARTEMIS-5871:
----------------------------------------------------------

Commit 9fe80fe5b0696370a830df1edf60fe9eaae14444 in artemis's branch 
refs/heads/main from Gary Tully
[ https://gitbox.apache.org/repos/asf?p=artemis.git;h=9fe80fe5b0 ]

ARTEMIS-5871 make it illegal to provide non xml reloadable config and a 
properties source when reload is enabled


> reload of broker properties config should be restricted to confined to 
> re-loadable components
> ---------------------------------------------------------------------------------------------
>
>                 Key: ARTEMIS-5871
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-5871
>             Project: Artemis
>          Issue Type: Bug
>          Components: Configuration
>    Affects Versions: 2.50.0
>            Reporter: Gary Tully
>            Assignee: Gary Tully
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 3h 40m
>  Remaining Estimate: 0h
>
> currently on reload config, broker properties are applied to the current 
> broker config in error. 
> This means that the absence of a value is not reflected in the config update, 
> simply removing the properties does not result in a removed component or 
> configuration entry.
> This is not consistent with the xml reload but also means for properties only 
> config, there needs to be an explicit remove key=- value. which then needs to 
> be removed. 
> If config reload of properties are confined to a new config that is then 
> compared in the normal way with the component reload logic, the properties 
> can be the source of truth.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to