You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cm...@apache.org on 2014/08/14 03:36:12 UTC
svn commit: r1617860 -
/hadoop/common/trunk/hadoop-tools/hadoop-gridmix/src/test/java/org/apache/hadoop/mapred/gridmix/TestGridmixRecord.java
Author: cmccabe
Date: Thu Aug 14 01:36:12 2014
New Revision: 1617860
URL: http://svn.apache.org/r1617860
Log:
HADOOP-10843. TestGridmixRecord unit tests failure on PowerPC (Jinghui Wang via Colin Patrick McCabe)
Modified:
hadoop/common/trunk/hadoop-tools/hadoop-gridmix/src/test/java/org/apache/hadoop/mapred/gridmix/TestGridmixRecord.java
Modified: hadoop/common/trunk/hadoop-tools/hadoop-gridmix/src/test/java/org/apache/hadoop/mapred/gridmix/TestGridmixRecord.java
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-tools/hadoop-gridmix/src/test/java/org/apache/hadoop/mapred/gridmix/TestGridmixRecord.java?rev=1617860&r1=1617859&r2=1617860&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-tools/hadoop-gridmix/src/test/java/org/apache/hadoop/mapred/gridmix/TestGridmixRecord.java (original)
+++ hadoop/common/trunk/hadoop-tools/hadoop-gridmix/src/test/java/org/apache/hadoop/mapred/gridmix/TestGridmixRecord.java Thu Aug 14 01:36:12 2014
@@ -140,10 +140,10 @@ public class TestGridmixRecord {
final int chk = WritableComparator.compareBytes(
out1.getData(), 0, out1.getLength(),
out2.getData(), 0, out2.getLength());
- assertEquals(chk, x.compareTo(y));
- assertEquals(chk, cmp.compare(
+ assertEquals(Integer.signum(chk), Integer.signum(x.compareTo(y)));
+ assertEquals(Integer.signum(chk), Integer.signum(cmp.compare(
out1.getData(), 0, out1.getLength(),
- out2.getData(), 0, out2.getLength()));
+ out2.getData(), 0, out2.getLength())));
// write second copy, compare eq
final int s1 = out1.getLength();
x.write(out1);
@@ -153,8 +153,8 @@ public class TestGridmixRecord {
y.write(out2);
assertEquals(0, cmp.compare(out2.getData(), 0, s2,
out2.getData(), s2, out2.getLength() - s2));
- assertEquals(chk, cmp.compare(out1.getData(), 0, s1,
- out2.getData(), s2, out2.getLength() - s2));
+ assertEquals(Integer.signum(chk), Integer.signum(cmp.compare(out1.getData(), 0, s1,
+ out2.getData(), s2, out2.getLength() - s2)));
}
}