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

Sam Tunnicliffe commented on CASSANDRA-13964:
---------------------------------------------

[~jasobrown] thanks for the review & sorry for the delay in getting back to 
this. 

You're totally right about the dtest, [so I've made it sane as you 
suggested|https://github.com/beobal/cassandra-dtest/commit/edc48bc965e842628413cfd50a7a21071d7b098a].
 As the upstream branches have moved on in since you reviewed, I've rebased & 
force pushed to trigger CI to run again. The addition of CASSANDRA-13291 to 
trunk was no issue as both branches took the same approach. Unless you have any 
concerns, I'll commit once CI is done. Thanks.

||branch||utest||dtest||
|[13964-3.0|https://github.com/beobal/cassandra/tree/13964-3.0]|[utest|https://circleci.com/gh/beobal/cassandra/tree/13964-3.0]|[dtest|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-dtest/401/]|
|[13964-3.11|https://github.com/beobal/cassandra/tree/13964-3.11]|[utest|https://circleci.com/gh/beobal/cassandra/tree/13964-3.11]|[dtest|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-dtest/402/]|
|[13964-trunk|https://github.com/beobal/cassandra/tree/13964-trunk]|[utest|https://circleci.com/gh/beobal/cassandra/tree/13964-trunk]|[dtest|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-dtest/403/]|

> Tracing interferes with digest requests when using RandomPartitioner
> --------------------------------------------------------------------
>
>                 Key: CASSANDRA-13964
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-13964
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Local Write-Read Paths, Observability
>            Reporter: Sam Tunnicliffe
>            Assignee: Sam Tunnicliffe
>
> A {{ThreadLocal<MessageDigest>}} is used to generate the MD5 digest when a 
> replica serves a read command and the {{isDigestQuery}} flag is set. The same 
> threadlocal is also used by {{RandomPartitioner}} to decorate partition keys. 
> So in a cluster with RP, if tracing is enabled the data digest is corrupted 
> by the partitioner making tokens for the tracing mutations. This causes a 
> digest mismatch on the coordinator, triggering a full data read on every read 
> where CL > 1 (or speculative execution/read repair kick in).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to