DocumentStore: StableRevisionComparator ignored the branch flag

[Thomas Mueller]

Hi,

I wonder if there is a bug in the StableRevisionComparator. It uses
Revision.compareRevisionTimeThenClusterId, which doesn't take the "branch"
flag into account. I would probably use the following instead:

/**
 * Compare all components of two revisions.
 * 
 * @param other the other revision
 * @return -1, 0, or 1
 */
    int compareTo(Revision other) {
    int comp = compareRevisionTimeThenClusterId(other);
    if (comp == 0) {
        if (branch != other.branch) {
            return branch ? -1 : 1;
        }
    }
    return comp;
    }



Regards,
Thomas