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

    https://github.com/apache/nifi/pull/2514#discussion_r172519579
  
    --- Diff: 
nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/test/java/org/apache/nifi/processors/mqtt/common/MqttTestClient.java
 ---
    @@ -17,16 +17,7 @@
     
     package org.apache.nifi.processors.mqtt.common;
     
    -import org.eclipse.paho.client.mqttv3.IMqttClient;
    -import org.eclipse.paho.client.mqttv3.IMqttDeliveryToken;
    -import org.eclipse.paho.client.mqttv3.IMqttToken;
    -import org.eclipse.paho.client.mqttv3.MqttCallback;
    -import org.eclipse.paho.client.mqttv3.MqttConnectOptions;
    -import org.eclipse.paho.client.mqttv3.MqttException;
    -import org.eclipse.paho.client.mqttv3.MqttMessage;
    -import org.eclipse.paho.client.mqttv3.MqttPersistenceException;
    -import org.eclipse.paho.client.mqttv3.MqttSecurityException;
    -import org.eclipse.paho.client.mqttv3.MqttTopic;
    +import org.eclipse.paho.client.mqttv3.*;
    --- End diff --
    
    please avoid star import.  Be sure to run 
    
    mvn clean install -Pcontrib-check


---

Reply via email to