Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/966#issuecomment-127415936
Hi @sachingoel0101,
thanks for opening this PR!
I see your point that users would have to implement `setRuntimeContext()`
and `getRuntimeContext()` if we would use an interface. However, I am not sure
that this would affect many people because we could implement the methods in
all our base Input/OutputFormats which should cover a large portion of new
formats. Also, these methods can be implemented in less than a minute. I think
@StephanEwen has a point that all Input/OutputFormats are always quite rich. I
would opt for the interface solution but implement it for all relevant
Input/OutputFormats that Flink provides.
I would also remove the `postConfigure()` method in favor of a clear
documentation of the interface stating that the `RuntimeContext`is available in
`open()`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
