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

    https://github.com/apache/spark/pull/19181#discussion_r141843914
  
    --- Diff: 
core/src/test/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorterSuite.java
 ---
    @@ -19,10 +19,18 @@
     
     import java.io.File;
     import java.io.IOException;
    +import java.io.PrintWriter;
     import java.util.Arrays;
     import java.util.LinkedList;
     import java.util.UUID;
     
    +import jodd.io.StringOutputStream;
    +import org.hamcrest.Description;
    +import org.hamcrest.Matcher;
    +import org.hamcrest.Matchers;
    +import org.hamcrest.TypeSafeMatcher;
    +import org.junit.Rule;
    +import org.junit.rules.ExpectedException;
    --- End diff --
    
    nit: I think you don't use most of these imports anymore.


---

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

Reply via email to