Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2450
I think these changes are good.
As a very personal and biased statement: I think the code overdoes it a bit
with the "every parameter on a new line" policy. We have beautiful wide screens
and I get sore fingers from scrolling up and down again to figure out enough
context of a code passage ;-)
Also, the style breaks the "visual" separation between different parts of
statement, like "members of a return tuple" vs. "function parameters", or
"clauses of an if statement" and "body of the if statement". I know the idea of
that pattern was to improve code readability, but I am getting the feeling this
is approaching a "verschlimmbesserung" (an improvement that makes things worse).
---
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.
---
