Github user pnowojski commented on a diff in the pull request:
https://github.com/apache/flink/pull/6392#discussion_r204695915
--- 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 --
Wasn't that a dead code? `CompositeTypeSerializerConfigSnapshot(null)`
resolves to `CompositeTypeSerializerConfigSnapshot(new TypeSerializer<>{null})`.
---
