[ 
https://issues.apache.org/jira/browse/CASSANDRA-14772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16623766#comment-16623766
 ] 

Aleksey Yeschenko commented on CASSANDRA-14772:
-----------------------------------------------

[~eperott] Yeah. It was very odd to find audit-log print-out code hidden in an 
if-else branch of a tool called {{fqltool dump}}, so I removed it - with an 
expectation that the missing functionality would be added *properly* at some 
point later. A bit surprised it's throwing an NPE though - I think it's 
supposed to throw an {{UnsupportedOperationException}} there, but it looks like 
Marcus is on it here.

> Refactor audit / full query log
> -------------------------------
>
>                 Key: CASSANDRA-14772
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-14772
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Marcus Eriksson
>            Assignee: Marcus Eriksson
>            Priority: Major
>             Fix For: 4.0
>
>
> I find the audit + full query log code quite hard to follow with some of the 
> naming and mixing of concepts, this ticket aims to:
> * move full query log specific code to a separate package 
> * do some audit log class renames (I keep reading {{BinLogAuditLogger}} vs 
> {{BinAuditLogger}} wrong for example)
> * avoid parsing the CQL queries twice in {{QueryMessage}} when audit log is 
> enabled.
> * add a new tool to dump audit logs (ie, let fqltool be full query log 
> specific)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to