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

Robert Metzger commented on FLINK-655:
--------------------------------------

I'm not sure if we still want this renaming. The discussion happened before the 
0.5 release. If we change this, our users have to change their code with the 
0.6 release.
The package renaming from Stratosphere to Flink is not a good argument here, 
because thats just a replacement of the imports.

> Rename DataSet.withBroadcastSet(DataSet, String) method
> -------------------------------------------------------
>
>                 Key: FLINK-655
>                 URL: https://issues.apache.org/jira/browse/FLINK-655
>             Project: Flink
>          Issue Type: Improvement
>            Reporter: Ufuk Celebi
>              Labels: 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.2#6252)

Reply via email to