[GitHub] nifi issue #2637: NIFI-5085 In InvokeHttp, moving the OkHttp Response object...

2018-04-18 Thread JPercivall
Github user JPercivall commented on the issue: https://github.com/apache/nifi/pull/2637 Will close, thanks for the review @mattyb149! For linkage: https://github.com/apache/nifi/commit/a0c9bebe241e8960363490cc0378f22d034f8db7 ---

[GitHub] nifi issue #2637: NIFI-5085 In InvokeHttp, moving the OkHttp Response object...

2018-04-18 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/2637 @JPercivall I forgot to sign off and put the "This closes" on the commit, do you mind closing this PR? Please and thanks and sorry about that! ---

[GitHub] nifi issue #2637: NIFI-5085 In InvokeHttp, moving the OkHttp Response object...

2018-04-18 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/2637 +1 LGTM, ran build with unit tests. Thanks for the improvement! Merging to master ---