[GitHub] [zookeeper] arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2019-10-11 Thread GitBox
arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers. URL: https://github.com/apache/zookeeper/pull/1105#issuecomment-540942744 > ... because 3.5 is already locked down for new features and release process of 3.6 has already been started. I think it

[GitHub] [zookeeper] arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2019-10-11 Thread GitBox
arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers. URL: https://github.com/apache/zookeeper/pull/1105#issuecomment-540943266 > In terms of the implementation: would you please make the `ZKAuditLogger` implementation pluggable similarly to `zookeeper

[GitHub] [zookeeper] arshadmohammad edited a comment on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2019-10-11 Thread GitBox
arshadmohammad edited a comment on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers. URL: https://github.com/apache/zookeeper/pull/1105#issuecomment-540942744 > ... because 3.5 is already locked down for new features and release process of 3.6 has already been started.

[GitHub] [zookeeper] iamgd67 opened a new pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources

2019-10-11 Thread GitBox
iamgd67 opened a new pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources URL: https://github.com/apache/zookeeper/pull/1114 zooinspector will start up with blank window when directly use IDE to run ZooInspector#main. this is caused by icos resource load faile

[GitHub] [zookeeper] eolivelli commented on issue #1114: ZOOKEEPER-3572: correctly handle zooinspector resources

2019-10-11 Thread GitBox
eolivelli commented on issue #1114: ZOOKEEPER-3572: correctly handle zooinspector resources URL: https://github.com/apache/zookeeper/pull/1114#issuecomment-540967429 great to see an active user of such tool. It would be awesome to see some feedback on the user/dev mailing list -

[GitHub] [zookeeper] eolivelli commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources

2019-10-11 Thread GitBox
eolivelli commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources URL: https://github.com/apache/zookeeper/pull/1114#discussion_r334012332 ## File path: zookeeper-contrib/zookeeper-contrib-zooinspector/pom.xml ## @@ -96,4 +96,15

[GitHub] [zookeeper] eolivelli commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources

2019-10-11 Thread GitBox
eolivelli commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources URL: https://github.com/apache/zookeeper/pull/1114#discussion_r334012332 ## File path: zookeeper-contrib/zookeeper-contrib-zooinspector/pom.xml ## @@ -96,4 +96,15

[GitHub] [zookeeper] iamgd67 commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources

2019-10-11 Thread GitBox
iamgd67 commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources URL: https://github.com/apache/zookeeper/pull/1114#discussion_r334213540 ## File path: zookeeper-contrib/zookeeper-contrib-zooinspector/pom.xml ## @@ -96,4 +96,15 @

[GitHub] [zookeeper] iamgd67 commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources

2019-10-11 Thread GitBox
iamgd67 commented on a change in pull request #1114: ZOOKEEPER-3572: correctly handle zooinspector resources URL: https://github.com/apache/zookeeper/pull/1114#discussion_r334213934 ## File path: zookeeper-contrib/zookeeper-contrib-zooinspector/pom.xml ## @@ -96,4 +96,15 @