[ https://issues.apache.org/jira/browse/CASSANDRA-15533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17105687#comment-17105687 ]
Jordan West commented on CASSANDRA-15533: ----------------------------------------- Since CASSANDRA-15392 is merged and this has existed for a while my preference would be to wait until the subsequent release. The improvement is minor if 15392 isn't merged and if it is merged only affects the changes if SASI is enabled. > 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