GitHub user mmolimar opened a pull request:

    https://github.com/apache/spark/pull/22383

    [SPARK-25395][JavaAPI] Removing Optional Spark Java API

    ## What changes were proposed in this pull request?
    
    Previous Spark versions didn't require Java 8 and an ``Optional`` Spark 
Java API had to be  implemented to support optional values.
    
    Since Spark 2.4 uses Java 8, the ``Optional`` Spark Java API should be 
removed so that Spark uses the original Java API.
    
    ## How was this patch tested?
    
    ``OptionalSuite`` class was removed to test Spark Java API ``Optional`` 
class (this class as well).
    Notice that the ``get`` method in the Spark Java API ``Optional`` class 
throws a ``NullPointerException`` when the value is not set whereas the native 
Java API ``java.util.Optional`` throws a ``NoSuchElementException``.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mmolimar/spark SPARK-25395

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/22383.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #22383
    
----
commit 4daf0ece245a2dc640be217cf4ad481ea430f996
Author: Mario Molina <mmolimar@...>
Date:   2018-09-10T14:48:13Z

    Removing Optional Spark Java API

----


---

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

Reply via email to