Github user hvanhovell commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19181#discussion_r143771647
  
    --- Diff: 
core/src/test/java/org/apache/spark/util/collection/unsafe/sort/UnsafeInMemorySorterSuite.java
 ---
    @@ -139,4 +139,49 @@ public int compare(
         }
         assertEquals(dataToSort.length, iterLength);
       }
    +
    +  @Test
    +  public void freeAfterOOM() {
    +    final SparkConf sparkConf =
    +            new SparkConf()
    --- End diff --
    
    NIT style


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to