[ https://issues.apache.org/jira/browse/CASSANDRA-12151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16416952#comment-16416952 ]
Vinay Chella commented on CASSANDRA-12151: ------------------------------------------ Hi [~djoshi3] [~jolynch], Thanks for the review, implemented review suggestions provided and pushed those changes in PR. \\ ||[branch|https://github.com/vinaykumarchella/cassandra/tree/trunk_CASSANDRA-12151]|| |[PR for trunk|https://github.com/vinaykumarchella/cassandra/pull/2/commits]| |[circleci|https://circleci.com/gh/vinaykumarchella/cassandra/tree/trunk_CASSANDRA-12151]| \\ {quote}Initialization can be simplified like this - excludedUsers = ImmutableSet.of(DatabaseDescriptor.getAuditLoggingOptions().excluded_users.split(",")); {quote} {{loadFilters()}} are designed so that filter values can be loaded during the startup as well as JMX call to reload the filters {quote}AuditLogFilter:: isFiltered - You can return directly at line 137 and get rid of isExcluded variable You can get rid of isIncluded as well and return at line 148. Let the control fall through to line 153 and it will return false. Furthermore, If you check the sets during initialization and ensure that they're mutually exclusive, your logic simplifies to if (includedSet.contains(object)) return false; else if (excludedSet.contains(object)}) return true; {quote} I went with the suggestions provided in former part to simplify the logic in {{isFiltered}} rather than mutual exclusion at initialization part. > Audit logging for database activity > ----------------------------------- > > Key: CASSANDRA-12151 > URL: https://issues.apache.org/jira/browse/CASSANDRA-12151 > Project: Cassandra > Issue Type: New Feature > Reporter: stefan setyadi > Assignee: Vinay Chella > Priority: Major > Fix For: 4.x > > Attachments: 12151.txt, CASSANDRA_12151-benchmark.html, > DesignProposal_AuditingFeature_ApacheCassandra_v1.docx > > > we would like a way to enable cassandra to log database activity being done > on our server. > It should show username, remote address, timestamp, action type, keyspace, > column family, and the query statement. > it should also be able to log connection attempt and changes to the > user/roles. > I was thinking of making a new keyspace and insert an entry for every > activity that occurs. > Then It would be possible to query for specific activity or a query targeting > a specific keyspace and column family. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org