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

Dinesh Joshi commented on CASSANDRA-15657:
------------------------------------------

Sure, IIRC the SSTable metadata only contains the beginning and ending tokens. 
However, in the wrap around use-case you'd need to unwrap the ranges and do the 
check. I don't think your check works in the wrap around case.

> Improve zero-copy-streaming containment check by using file sections
> --------------------------------------------------------------------
>
>                 Key: CASSANDRA-15657
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15657
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Legacy/Streaming and Messaging
>            Reporter: ZhaoYang
>            Assignee: ZhaoYang
>            Priority: Normal
>             Fix For: 4.0
>
>
> Currently zero copy streaming is only enabled for leveled-compaction strategy 
> and it checks if all keys in the sstables are included in the transferred 
> ranges.
> This is very inefficient. The containment check can be improved by checking 
> if transferred sections (the transferred file positions) cover entire sstable.
> I also enabled ZCS for all compaction strategies since the new containment 
> check is very fast..



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to