[
https://issues.apache.org/jira/browse/FLINK-655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14281850#comment-14281850
]
Stephan Ewen commented on FLINK-655:
------------------------------------
Admittedly, there is a slight naming inconsistency. But I think that at this
point, API stability is more important.
I am not really in favor of breaking things for minor cosmetic corrections.
The original rational behind the naming was: *withBroadcastSet*, when adding a
data set to be broadcasted. *getBroadcastVariable*, because at this point it is
actually not some monadic data set structure any more, but a concrete locally
materialized variable.
> Add support for both single and set of broadcast values
> -------------------------------------------------------
>
> Key: FLINK-655
> URL: https://issues.apache.org/jira/browse/FLINK-655
> Project: Flink
> Issue Type: Improvement
> Components: Java API
> Reporter: Ufuk Celebi
> Assignee: Henry Saputra
> Labels: breaking-api, github-import, starter
> Fix For: pre-apache
>
>
> To broadcast a data set you have to do the following:
> ```java
> lorem.map(new MyMapper()).withBroadcastSet(toBroadcast, "toBroadcastName")
> ```
> In the operator you call:
> ```java
> getRuntimeContext().getBroadcastVariable("toBroadcastName")
> ```
> I propose to have both method names consistent, e.g.
> - `withBroadcastVariable(DataSet, String)`, or
> - `getBroadcastSet(String)`.
> ---------------- Imported from GitHub ----------------
> Url: https://github.com/stratosphere/stratosphere/issues/655
> Created by: [uce|https://github.com/uce]
> Labels: enhancement, java api, user satisfaction,
> Created at: Wed Apr 02 16:29:08 CEST 2014
> State: open
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
