[GitHub] nifi issue #2663: NIFI-5122 - Add Record Writer for S2S RTs

2018-04-27 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/2663 Reviewing... ---

[GitHub] nifi issue #2663: NIFI-5122 - Add Record Writer for S2S RTs

2018-04-27 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/2663 Thanks for the feedback @mattyb149, I just pushed another commit, let me know if it's what you had in mind :) ---

[GitHub] nifi issue #2663: NIFI-5122 - Add Record Writer for S2S RTs

2018-05-25 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2663 @pvillard31 @mattyb149 Reviewed it and the changes look fine, but noticed something weird when I fired up a full build w/ it. When I'd add one of these tasks, the status would stay spinning at "va

[GitHub] nifi issue #2663: NIFI-5122 - Add Record Writer for S2S RTs

2018-05-25 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/2663 @MikeThomsen - I know @mcgilman recently fixed something weird in the UI around CS/RT dialogs. Could you try cleaning your cache or using a private session to confirm? ---

[GitHub] nifi issue #2663: NIFI-5122 - Add Record Writer for S2S RTs

2018-05-25 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2663 Cleared the cache, and it didn't change anything. However, all three of the s2s tasks I tried (metrics, bulletin and provenance) seemed to work just fine when reporting to a second instance of NiF

[GitHub] nifi issue #2663: NIFI-5122 - Add Record Writer for S2S RTs

2018-05-29 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2663 @pvillard31 @mattyb149 I'll merge today if one of you gets back to me on this. ---