Github user jbertram commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1990#discussion_r178924323
  
    --- Diff: 
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionImpl.java
 ---
    @@ -29,12 +29,7 @@
     import java.util.concurrent.atomic.AtomicInteger;
     
     import org.apache.activemq.artemis.api.config.ActiveMQDefaultConfiguration;
    -import org.apache.activemq.artemis.api.core.ActiveMQBuffer;
    -import org.apache.activemq.artemis.api.core.ActiveMQBuffers;
    -import org.apache.activemq.artemis.api.core.ActiveMQException;
    -import org.apache.activemq.artemis.api.core.ActiveMQExceptionType;
    -import org.apache.activemq.artemis.api.core.Message;
    -import org.apache.activemq.artemis.api.core.SimpleString;
    +import org.apache.activemq.artemis.api.core.*;
    --- End diff --
    
    Star imports like this are not allowed by our checkstyle config. You'll 
need to change this for the PR build to pass.


---

Reply via email to