cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedict (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-9988) Introduce leaf-only iterator
Date Wed, 25 Jan 2017 18:27:26 GMT

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

Benedict commented on CASSANDRA-9988:
-------------------------------------

Sorry Jay, not anytime soon.  I no longer work actively on the project and am rather busy;
you should look for one of the full-time contributors to lend a hand getting this merged.

I took a quick peek, and I'll note (without commenting on the quality of the code itself)
for any potential reviewers out there that the patch looks to be in a good shape generally,
so I shouldn't expect it to be an onerous review.

I did notice that in LeafBtree... line 108-109 are probably redundant, assuming line 106 does
its job

> Introduce leaf-only iterator
> ----------------------------
>
>                 Key: CASSANDRA-9988
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9988
>             Project: Cassandra
>          Issue Type: Sub-task
>            Reporter: Benedict
>            Assignee: Jay Zhuang
>            Priority: Minor
>              Labels: patch
>             Fix For: 4.0
>
>         Attachments: 9988-trunk-new.txt, trunk-9988.txt
>
>
> In many cases we have small btrees, small enough to fit in a single leaf page. In this
case it _may_ be more efficient to specialise our iterator.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message