Github user tzulitai commented on a diff in the pull request:
https://github.com/apache/flink/pull/6392#discussion_r204684670
--- Diff:
flink-core/src/main/java/org/apache/flink/api/common/typeutils/CompositeTypeSerializerConfigSnapshot.java
---
@@ -46,8 +46,6 @@
public CompositeTypeSerializerConfigSnapshot() {}
public CompositeTypeSerializerConfigSnapshot(TypeSerializer<?>...
nestedSerializers) {
- Preconditions.checkNotNull(nestedSerializers);
--- End diff --
Not sure about this. It should be perfectly fine that we have a null check
here.
---
