[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-12 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2206 I discovered some issues with the admin toolkit and will be opening a Jira to resolve that. In the meantime, I validated everything except the toolkit behavior for this PR, but that was confirmed by

[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-11 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2206 I had some trouble getting the cluster configured successfully due to Zookeeper port conflicts (will be filing a Jira to improve the error messaging there). I got it working and the flow operates su

[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-11 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/2206 Thanks @bbende. I've pushed another commit that addresses the checkstyle issues. ---

[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-11 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2206 Reviewing... ---

[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-11 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/2206 Also some unused imports in the Yandex bundle ---

[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-11 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/2206 There is a minor check style violation in RedirectResourceFilter for the @return java doc. ---

[GitHub] nifi issue #2206: NIFI-4444: Upgrade to Jersey 2.x

2017-10-11 Thread mcgilman
Github user mcgilman commented on the issue: https://github.com/apache/nifi/pull/2206 This PR upgrades Jersey throughout NiFi where it is used directly. Most transitive Jersey dependencies are left intact. The most significant changes are centered in the clustering framework, UpdateAt