Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2039#discussion_r184178629
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ActiveMQServerImpl.java
---
@@ -2323,6 +2324,16 @@ synchronized boolean initialisePart1(boolean
scalingDown) throws Exception {
deployAddressSettingsFromConfiguration();
}
+ //fix of ARTEMIS-1823
+ if (!configuration.isPersistenceEnabled()) {
+ for (AddressSettings addressSettings :
addressSettingsRepository.values()) {
+ if (addressSettings.getAddressFullMessagePolicy() ==
AddressFullMessagePolicy.PAGE) {
+ ActiveMQServerLogger.LOGGER.pageWillBePersisted();
--- End diff --
I just don't think it's necessarily a Warning.. it's a feature!
I don't agree to make this a WARN.. users using this on purpose will get a
nonSense warning.
if you want to log about the condition, I would make it INFO. as this is an
intended feature.
---
