[
https://issues.apache.org/jira/browse/FLINK-656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ufuk Celebi resolved FLINK-656.
-------------------------------
Resolution: Won't Fix
OK, I didn't want to kick off an endless discussion.
The only advantage I saw was the following: with it in the open method, it
would be "self documenting" in the sense that new users wouldn't have to know
that there is a `getRuntimeContext()` for user functions.
> Add RuntimeContext to open method of functions
> ----------------------------------------------
>
> Key: FLINK-656
> URL: https://issues.apache.org/jira/browse/FLINK-656
> Project: Flink
> Issue Type: Bug
> Components: Java API
> Reporter: Ufuk Celebi
> Labels: breaking-api, github-import
> Fix For: pre-apache
>
>
> This is minor.
> Is it reasonable to add the `RuntimeContext` as an argument to the
> `open(Configuration)` method of stub functions? Can we do this without
> breaking stuff?
> I would find it nice to replace the possibly repeated calls to
> `getRuntimeContext().getBroadcastVariable()`,
> `getRuntimeContext().addAccumulator()`, etc. with something like
> `context.getBroadcastVariable()`.
> ---------------- Imported from GitHub ----------------
> Url: https://github.com/stratosphere/stratosphere/issues/656
> Created by: [uce|https://github.com/uce]
> Labels:
> Created at: Wed Apr 02 16:33:51 CEST 2014
> State: open
--
This message was sent by Atlassian JIRA
(v6.2#6252)
