spark-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Markus Weimer (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (SPARK-4819) Remove Guava's "Optional" from public API
Date Wed, 06 Jan 2016 19:10:39 GMT

    [ 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


Mime
View raw message