[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2018-09-17 Thread arshadmohammad
Github user arshadmohammad commented on the issue: https://github.com/apache/zookeeper/pull/338 @pmoust I was pointing to comments: [1](https://issues.apache.org/jira/browse/ZOOKEEPER-1260?focusedCommentId=16612302&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#c

[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2018-09-17 Thread pmoust
Github user pmoust commented on the issue: https://github.com/apache/zookeeper/pull/338 @arshadmohammad hm - they don't seem to be any(?) ---

[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2018-09-17 Thread arshadmohammad
Github user arshadmohammad commented on the issue: https://github.com/apache/zookeeper/pull/338 Hi all, please find performance impact analysis in jira comments. ---

[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2018-06-25 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/338 @arshadmohammad Can we move on with this great new feature? ---

[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2018-04-23 Thread pmoust
Github user pmoust commented on the issue: https://github.com/apache/zookeeper/pull/338 @arshadmohammad I'd also be curious as to the perf implications of this, have you run a load test-suite against this? ---

[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2017-09-11 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/338 @arshadmohammad Audit logging, when enabled, will for sure degrade performance as it adds additional IO cost. Have you done any sort of performance evaluation so we can roughly get an idea of what ki

[GitHub] zookeeper issue #338: ZOOKEEPER-1260:Audit logging in ZooKeeper servers.

2017-09-04 Thread arshadmohammad
Github user arshadmohammad commented on the issue: https://github.com/apache/zookeeper/pull/338 Thanks @hanm @afine for reviewing this feature. I have addressed all the comments, Please have a look. ---