You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Markus Weimer (JIRA)" <ji...@apache.org> on 2016/01/06 20:10:39 UTC

[jira] [Comment Edited] (SPARK-4819) Remove Guava's "Optional" from public API

    [ https://issues.apache.org/jira/browse/SPARK-4819?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15086083#comment-15086083 ] 

Markus Weimer edited comment on SPARK-4819 at 1/6/16 7:09 PM:
--------------------------------------------------------------

Over in REEF, we just backported Java 8's {{Optional<T>}} class. Would that be an option for Spark as well? If so, maybe this could go to one of the commons packages such that we don't clone the same trivial class over and over again :-)


was (Author: markus.weimer):
Over in REEF, we just backportd Java 8's [[Optional<T>]] class. Would that be an option for Spark as well? If so, maybe this could go to one of the commons packages such that we don't clone the same trivial class over and over again :-)

> Remove Guava's "Optional" from public API
> -----------------------------------------
>
>                 Key: SPARK-4819
>                 URL: https://issues.apache.org/jira/browse/SPARK-4819
>             Project: Spark
>          Issue Type: Sub-task
>          Components: Spark Core
>    Affects Versions: 1.2.0
>            Reporter: Marcelo Vanzin
>            Assignee: Sean Owen
>         Attachments: SPARK_4819_null_do_not_merge.patch
>
>
> Filing this mostly so this isn't forgotten. Spark currently exposes Guava types in its public API (the {{Optional}} class is used in the Java bindings). This makes it hard to properly hide Guava from user applications, and makes mixing different Guava versions with Spark a little sketchy (even if things should work, since those classes are pretty simple in general).
> Since this changes the public API, it has to be done in a release that allows such breakages. But it would be nice to at least have a transition plan for deprecating the affected APIs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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