You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by srowen <gi...@git.apache.org> on 2018/10/10 20:24:12 UTC

[GitHub] spark pull request #22383: [SPARK-25395][JavaAPI] Replace Spark Optional cla...

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

    https://github.com/apache/spark/pull/22383#discussion_r224228243
  
    --- Diff: core/src/test/java/test/org/apache/spark/JavaAPISuite.java ---
    @@ -476,10 +476,10 @@ public void leftOuterJoin() {
           new Tuple2<>(2, 'z'),
           new Tuple2<>(4, 'w')
         ));
    -    List<Tuple2<Integer,Tuple2<Integer,Optional<Character>>>> joined =
    +    List<Tuple2<Integer,Tuple2<Integer, Optional<Character>>>> joined =
    --- End diff --
    
    @mmolimar Could you revert these whitespace changes?


---

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