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

Per Otterström commented on CASSANDRA-15076:
--------------------------------------------

I've created a patch based on the proposal above.

[Patch|https://github.com/eperott/cassandra/tree/15076-trunk]
[Unit tests|https://circleci.com/gh/eperott/cassandra/66#tests/containers/2]
[FQL Unit tests|https://circleci.com/gh/eperott/cassandra/67]

A local run of the dtests result in some flaky failures, but nothing that seem 
related to fql or audit. In particular, the {{auditlog_test.py}} and 
{{fqltool_test.py}} run without errors.


> Align record header of FQL and audit binary log
> -----------------------------------------------
>
>                 Key: CASSANDRA-15076
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15076
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Legacy/Tools, Tool/fql
>            Reporter: Per Otterström
>            Assignee: Per Otterström
>            Priority: Normal
>             Fix For: 4.0
>
>
> The new full query logger and the audit logger support logging into binary 
> Chronicle logs. Both create records with a small header to indicate what 
> follows, but the two features have adopted different header formats. Let's 
> align the record header format with this ticket.
>  * Both features should use the same header layout. This makes it possible to 
> give more user friendly error messages in the {{fqltool}} and 
> {{auditlogviewr}} commands.
>  * The record header should have a distinct {{type}} to indicate the type of 
> record.
>  * The record header should have a {{version}} so that the record format can 
> evolve.
> Current record header format of the FQL is:
> {noformat}
> version:0(int16)
> type:<type>(text)
> {noformat}
> where {{<type>}} can be either {{batch}} or {{single-query}}.
> Current record header format of the binary audit log is:
> {noformat}
> type:AuditLog(text)
> {noformat}



--
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

Reply via email to