Github user uce commented on the issue:
https://github.com/apache/flink/pull/3368
Hey Stephan! The changes look very good. Thanks also for your explanations,
I think this is something that we should focus on more during code reviews.
Actually, a section in the Internals or Contribution docs or Wiki would be
helpful.
A question by example:
Curator `throws Exception` on most operations. Previously, this was simply
forwarded. Should this now be wrapped in a `FlinkRuntimeException`?
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
