Airblader commented on a change in pull request #15369:
URL: https://github.com/apache/flink/pull/15369#discussion_r601247152
##########
File path:
flink-table/flink-table-runtime-blink/src/main/java/org/apache/flink/table/runtime/typeutils/ExternalSerializer.java
##########
@@ -59,15 +59,19 @@
private final TypeSerializer<I> internalSerializer;
+ private final boolean bidirectional;
Review comment:
I went with a simple flag to start with as a prototype. But we should
discuss how we want to model this, boolean flags don't make for nice APIs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
