Github user jmesnil commented on the pull request:
https://github.com/apache/activemq-artemis/commit/4db8cd54ca2174357e36e4f0c231ad66f128d8b8#commitcomment-24878586
In
artemis-server/src/main/java/org/apache/activemq/artemis/core/deployers/impl/FileConfigurationParser.java:
In
artemis-server/src/main/java/org/apache/activemq/artemis/core/deployers/impl/FileConfigurationParser.java
on line 1737:
Maybe this should be discussed on the artemis mailing list as a general
topic but I very dislike putting *every setter on the same line* when using a
fluent API.
Next time a property will be added/removed/updated, the diff will be
absolutely impossible to review.
---
