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