You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Lars Hofhansl (Updated) (JIRA)" <ji...@apache.org> on 2011/11/16 19:22:52 UTC

[jira] [Updated] (HBASE-4800) Result.compareResults is incorrect

     [ https://issues.apache.org/jira/browse/HBASE-4800?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lars Hofhansl updated HBASE-4800:
---------------------------------

    Attachment: 4800.txt

Simple patch with test
                
> Result.compareResults is incorrect
> ----------------------------------
>
>                 Key: HBASE-4800
>                 URL: https://issues.apache.org/jira/browse/HBASE-4800
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4, 0.92.0, 0.94.0
>            Reporter: Lars Hofhansl
>         Attachments: 4800.txt
>
>
> A coworker of mine (James Taylor) found a bug in Result.compareResults(...).
> This condition:
> {code}
>       if (!ourKVs[i].equals(replicatedKVs[i]) &&
>           !Bytes.equals(ourKVs[i].getValue(), replicatedKVs[i].getValue())) {
>         throw new Exception("This result was different: "
> {code}
> should be
> {code}
>       if (!ourKVs[i].equals(replicatedKVs[i]) ||
>           !Bytes.equals(ourKVs[i].getValue(), replicatedKVs[i].getValue())) {
>         throw new Exception("This result was different: "
> {code}
> Just checked, this is wrong in all branches.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira