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

arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers. URL: https://github.com/apache/zookeeper/pull/1105#issuecomment-552758818 Feature is merged in PR https://github.com/apache/zookeeper/pull/1133 --

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

arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers. URL: https://github.com/apache/zookeeper/pull/1105#issuecomment-549922014 Addressed all the comments and created PR for master branch. https://github.com/apache/zookeeper/pull/1133 --

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

arshadmohammad commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers. URL: https://github.com/apache/zookeeper/pull/1105#issuecomment-549404116 @anmolnar, I will update the patch soon, hopefully by tomorrow. Will try to wind up this work by this weekend -

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

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 commented on issue #1105: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

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