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

Benjamin Lerer edited comment on CASSANDRA-15533 at 4/24/20, 12:48 PM:
-----------------------------------------------------------------------

The patch looks good to me too.


was (Author: blerer):
The patch looks good to me.

> Don't allocate unneeded MergeIterator in OnDiskToken#iterator 
> --------------------------------------------------------------
>
>                 Key: CASSANDRA-15533
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15533
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Feature/SASI
>            Reporter: Jordan West
>            Assignee: Jordan West
>            Priority: Normal
>             Fix For: 4.0, 4.0-rc
>
>
> When reviewing CASSANDRA-15392 it became apparent that the MergeIterator 
> allocated by OnDiskToken#iterator is rarely necessary and so we should avoid 
> allocating one when not needed and skip the MergeIterator pool when needed 
> because its unlikely to be sized correctly. 



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