Github user SebastianCarroll commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2154#discussion_r143336390
--- Diff:
nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/test/java/org/apache/nifi/processors/mqtt/common/MqttTestClient.java
---
@@ -195,4 +218,296 @@ public String getServerURI() {
public void close() throws MqttException {
}
+
--- End diff --
Hey @joewitt! The newer version adds methods to the IMqttClient interface
and I auto-imported them. It pulled in the JavaDocs as well (could be the IDEA
default?) and I originally left them as I was sure if they were required. I
have removed them in the new update as it seems unnecessary to include them for
the MqttTestClient. Would you agree?
---
