Github user josephxsxn commented on the issue:
https://github.com/apache/nifi/pull/1800
1) I'll look into taking the inner classes out but will need to look at how
the permissions scopes are working between them first.
2) Will look into applying the KeeperException recommendation
Github user joewitt commented on the issue:
https://github.com/apache/nifi/pull/1800
couple of quick thoughts.
- we'll need to get all the version numbers aligned with whatever nifi
version this would be committed into. Currently that would be 1.3.0-SNAPSHOT.
- It would
Github user GrayGwizdz commented on the issue:
https://github.com/apache/nifi/pull/1800
You might want to consider importing KeeperException so catch
(KeeperException.NodeExistsException e) doesn't need the class in multiple
places
You also might want to consider refactoring to