Github user jmesnil commented on the pull request:

    
https://github.com/apache/activemq-artemis/commit/4db8cd54ca2174357e36e4f0c231ad66f128d8b8#commitcomment-24878562
  
    In 
artemis-server/src/main/java/org/apache/activemq/artemis/core/config/TransformerConfiguration.java:
    In 
artemis-server/src/main/java/org/apache/activemq/artemis/core/config/TransformerConfiguration.java
 on line 23:
    Why is this class using a fluent API instead of setting the class name and 
the properties in its constructor?
    The fields are mutable and it puts the burden on the user of the object to 
check that the class name of the config is not null.
    It must not be possible to create a transformer configuration without a 
class name.


---

Reply via email to