[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi/pull/392 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enable

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-24 Thread olegz
Github user olegz commented on the pull request: https://github.com/apache/nifi/pull/392#issuecomment-221343071 +1, all is good, test are passing. You'll need another PR for 0.x branch since the commits can not be cherry picked cleanly. --- If your project is set up for it, you c

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on the pull request: https://github.com/apache/nifi/pull/392#issuecomment-220432007 @olegz pushed out a commit addressing the comments (sans the SSL Context Service property mapping, due to my comment above) --- If your project is set up for it, you c

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63941204 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63919624 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63920230 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63919932 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63919082 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63918814 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63918531 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63918463 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63917779 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63908853 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,237 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63907479 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63907025 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63906843 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63903305 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,338 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63902702 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,338 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63902327 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,338 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-19 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63900979 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,338 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on the pull request: https://github.com/apache/nifi/pull/392#issuecomment-219860709 @olegz just pushed out a new patch addressing the second round of comments and rebased to master to fix the merge conflicts --- If your project is set up for it, you c

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63608020 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63592512 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63585339 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63585228 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63585234 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63585043 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63585015 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63584936 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63584682 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63583956 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63583750 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63583614 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63582187 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63581917 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63581819 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63581403 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63580606 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63579264 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63580163 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63580288 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63580066 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63579787 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63579631 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63578922 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63578737 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63578415 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63577875 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63577324 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,239 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on the pull request: https://github.com/apache/nifi/pull/392#issuecomment-219808772 @JPercivall just a heads up; the branch now has merge conflicts --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63558222 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63551055 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63549263 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63547530 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63545947 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63544945 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63543066 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63540545 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63539712 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63539198 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63539083 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0 +1,3

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63537926 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63536744 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63537518 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63536461 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63536287 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63535091 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63533161 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63534793 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63533978 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63532252 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63530121 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/ConsumeMQTT.java --- @@ -0,0 +1,341 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-17 Thread JPercivall
Github user JPercivall commented on the pull request: https://github.com/apache/nifi/pull/392#issuecomment-219721262 @olegz I pushed out new commits which address your comments. Also I added a bit more functionality I noticed other MQTT projects expose. Lastly I overhauled the unit te

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-13 Thread mosermw
Github user mosermw commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r63224728 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-04 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r62033274 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/common/AbstractMQTTProcessor.java --- @@ -0,0

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61956949 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61954882 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61950939 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/*

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61949787 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/SubscribeMQTT.java --- @@ -0,0 +1,301 @@

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61949840 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/SubscribeMQTT.java --- @@ -0,0 +1,301 @@

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61949743 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/

[GitHub] nifi pull request: NIFI-1808 Creating general MQTT processors

2016-05-03 Thread JPercivall
Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/392#discussion_r61948441 --- Diff: nifi-nar-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/main/java/org/apache/nifi/processors/mqtt/PublishMQTT.java --- @@ -0,0 +1,288 @@ +/