twalthr commented on code in PR #20555:
URL: https://github.com/apache/flink/pull/20555#discussion_r945582946
##########
flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/plan/nodes/exec/ExecNodeBase.java:
##########
@@ -49,6 +49,13 @@
@JsonIgnoreProperties(ignoreUnknown = true)
public abstract class ExecNodeBase<T> implements ExecNode<T> {
+ /**
+ * The default value is chosen for the JSON deserialization case. Other
cases must set this flag
+ * accordingly via {@link #setCompiled(boolean)}. It is not exposed via a
constructor arg to
+ * avoid complex constructor overloading for all {@link ExecNode}s.
+ */
+ private transient boolean isCompiled = true;
Review Comment:
The more time you invest in this topic, the nicer the solution gets. I found
a better way using `@JsonInject`. I will merge this once the build turns green,
if you don't have any objections.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
