[HADOOP-10843] TestGridmixRecord unit tests failure on PowerPC

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.0, 2.3.0, 2.4.0, 2.4.1
• Fix Version/s: 2.6.0
• Component/s: test, tools
• Target Version/s: 2.6.0
• Assignee: Jinghui Wang
• Reporter: Jinghui Wang
• Created: 15/Jul/14 23:17
• Updated: 01/Dec/14 03:07
• Resolved: 14/Aug/14 01:36

Description

In TestGridmixRecord#binSortTest, the test expects the comparison result of WritableComparator.compareBytes, which uses UnsafeComparer, to be the integer difference rather than the documented "@return 0 if equal, < 0 if left is less than right, etc.".

TestGridmixRecord#binSortTest code snippet

      final int chk = WritableComparator.compareBytes(
          out1.getData(), 0, out1.getLength(),
          out2.getData(), 0, out2.getLength());
      assertEquals(chk, x.compareTo(y));
      assertEquals(chk, cmp.compare(
            out1.getData(), 0, out1.getLength(),
            out2.getData(), 0, out2.getLength()));

The code snippet below shows the Unsafe comparator behavior for non-little-endian machines.

	if (!littleEndian) {
	  return lessThanUnsigned(lw, rw) ? -1 : 1;
	}