[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread david-streamlio
Github user david-streamlio commented on the issue: https://github.com/apache/nifi/pull/3178 @joewitt What was the command you used to generate the error above? I want to attempt to reproduce it locally if possible. The basic issue is that since this test message was sent

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/3178 the output log had "14:11:44.805 [pool-67-thread-1] ERROR org.apache.nifi.processors.pulsar.pubsub.PublishPulsarRecord - PublishPulsarRecord[id=9c298cea-0e2a-4a59-8864-92e95563d7f4] Unable to publish

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/3178 build on large machine now has a unit test failure [ERROR] Failures: [ERROR] TestAsyncPublishPulsarRecord.pulsarClientExceptionTest:56 expected:<1> but was:<0> [INFO] [ERROR]

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/3178 cool thanks. can you please take a look at my comment regarding the security related property. ---

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread david-streamlio
Github user david-streamlio commented on the issue: https://github.com/apache/nifi/pull/3178 @joewitt I dug into the build logs and found that the issue with the parallel build was due to having the wrong version for both of the Pulsar NAR files defined in the NiFi-assembly/pom.xml

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/3178 just did a full clean build with 6 threads on an older macbook and no problems. i'll try to reproduce my build issue from yesterday on a much higher thread build/machine and advise if issue i saw

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread david-streamlio
Github user david-streamlio commented on the issue: https://github.com/apache/nifi/pull/3178 @joewitt I have incorporated all of the changes / corrections that came from both your comments as well as @pvillard31's

[GitHub] nifi issue #3178: NIFI-4914: Add Apache Pulsar processors

2018-11-20 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/3178 @david-streamlio i had comments on the previous PR. Can you please review those/pull them forward to this. In the future you could rebase, squash, and force push the PR if you want to get back to a