cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joshua McKenzie (JIRA)" <>
Subject [jira] [Updated] (CASSANDRA-13830) Simplify MerkleTree.difference/differenceHelper
Date Tue, 03 Oct 2017 13:56:00 GMT


Joshua McKenzie updated CASSANDRA-13830:
    Issue Type: Improvement  (was: Bug)

> Simplify MerkleTree.difference/differenceHelper
> -----------------------------------------------
>                 Key: CASSANDRA-13830
>                 URL:
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Blake Eggleston
>            Assignee: Blake Eggleston
> As brought up in CASSANDRA-13603, {{MerkleTree.differenceHelper}} is overly complex and
difficult to follow for what it's doing. It also shares some of it's responsibilities with
{{difference}}, and assumes that the trees it's given have differences, which makes it a potential
source of future bugs.
> Since we're just trying to recursively compare these trees and record the largest contiguous
out of sync ranges, I think this could be simplified a bit. I propose that we refactor {{difference}}
/ {{differenceHelper}} so that {{difference}} is only concerned with supplying the range,
and dealing with the {{FULLY_INCONSISTENT}} case, and move everything else into a recursable
helper method.
> I put together an alternate implementation [here|].

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message