gtully commented on a change in pull request #3904:
URL: https://github.com/apache/activemq-artemis/pull/3904#discussion_r781131342
##########
File path:
artemis-server/src/main/java/org/apache/activemq/artemis/core/config/impl/ConfigurationImpl.java
##########
@@ -429,8 +444,28 @@ public String getSystemPropertyPrefix() {
return systemPropertyPrefix;
}
+ public String getBrokerPropertiesSystemPropertyName() {
+ return brokerPropertiesSystemPropertyName;
+ }
+
+ public void setBrokerPropertiesSystemPropertyName(String
brokerPropertiesSystemPropertyName) {
+ this.brokerPropertiesSystemPropertyName =
brokerPropertiesSystemPropertyName;
+ }
+
@Override
- public Configuration parseSystemProperties() throws Exception {
+ public Configuration parseProperties() throws Exception {
+ final String propertiesConfigFileName =
System.getProperty(getBrokerPropertiesSystemPropertyName());
Review comment:
I am not sure of the benefit of multiple files, but the cli could
arrange for the system property to be initialised with a value from a
--properties argument for sure.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
