Fix off-by-one in nextInt() call

JoshRosen · JoshRosen · commit 0ad34da135f2 · 2015-05-12T12:05:59.000-07:00
diff --git a/core/src/test/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleInMemorySorterSuite.java b/core/src/test/java/org/apache/spark/shuffle/unsafe/UnsafeShuffleInMemorySorterSuite.java
@@ -115,7 +115,7 @@ public void testSortingManyNumbers() throws Exception {
     int[] numbersToSort = new int[128000];
     Random random = new Random(16);
     for (int i = 0; i < numbersToSort.length; i++) {
-      numbersToSort[i] = random.nextInt(PackedRecordPointer.MAXIMUM_PARTITION_ID);
+      numbersToSort[i] = random.nextInt(PackedRecordPointer.MAXIMUM_PARTITION_ID + 1);
       sorter.insertRecord(0, numbersToSort[i]);
     }
     Arrays.sort(numbersToSort);

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ public void testSortingManyNumbers() throws Exception {`
`115`	`115`	`int[] numbersToSort = new int[128000];`
`116`	`116`	`Random random = new Random(16);`
`117`	`117`	`for (int i = 0; i < numbersToSort.length; i++) {`
`118`		`- numbersToSort[i] = random.nextInt(PackedRecordPointer.MAXIMUM_PARTITION_ID);`
	`118`	`+ numbersToSort[i] = random.nextInt(PackedRecordPointer.MAXIMUM_PARTITION_ID + 1);`
`119`	`119`	`sorter.insertRecord(0, numbersToSort[i]);`
`120`	`120`	`}`
`121`	`121`	`Arrays.sort(numbersToSort);`