Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/commit/1f60a1de563ccca4ea0309fbd5c6c3531090ddc9#commitcomment-26715572
In
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/api/serialization/SpanningRecordSerializer.java:
In
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/api/serialization/SpanningRecordSerializer.java
on line 62:
By common style, constructors use `this.` for all parameters. I would vote
to uphold that style.
---
