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

    https://github.com/apache/spark/pull/22399#discussion_r216917780
  
    --- Diff: 
common/kvstore/src/test/java/org/apache/spark/util/kvstore/DBIteratorSuite.java 
---
    @@ -383,7 +383,7 @@ public void testRefWithIntNaturalKey() throws Exception 
{
         LevelDBSuite.IntKeyType i = new LevelDBSuite.IntKeyType();
         i.key = 1;
         i.id = "1";
    -    i.values = Arrays.asList("1");
    +    i.values = Collections.singletonList("1");
    --- End diff --
    
    I agree, I don't really care about moving to `Collections`.


---

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

Reply via email to