Github user alopresto commented on the issue:
https://github.com/apache/nifi/pull/1421
Reviewed, reviewed reference documentation provided by @trixpan , and ran
all tests and contrib-check. Merged.
---
If your project is set up for it, you can reply to this email and have your
Github user alopresto commented on the issue:
https://github.com/apache/nifi/pull/1421
Reviewing...
---
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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1421
@joewitt
sure thing:
https://github.com/elastic/logstash-forwarder/blob/master/PROTOCOL.md#ack-frame-type
---
If your project is set up for it, you can reply to this email and
Github user joewitt commented on the issue:
https://github.com/apache/nifi/pull/1421
@trixpan is there anything we can reference for this change showing it
should be an int instead of a long? It is clearly a pretty simple change just
want to have some clear basis for it. thanks
