Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22730#discussion_r225286458
--- Diff: core/src/test/java/test/org/apache/spark/JavaAPISuite.java ---
@@ -186,7 +184,7 @@ public void randomSplit() {
long s1 = splits[1].count();
long s2 = splits[2].count();
assertTrue(s0 + " not within expected range", s0 > 150 && s0 < 250);
- assertTrue(s1 + " not within expected range", s1 > 250 && s0 < 350);
+ assertTrue(s1 + " not within expected range", s1 > 250 && s1 < 350);
--- End diff --
This was a fix I spotted, along with a few others, while modifying this
test. Not strictly related.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
