Zakelly commented on code in PR #26120:
URL: https://github.com/apache/flink/pull/26120#discussion_r1947412905
##########
flink-core/src/main/java/org/apache/flink/api/common/state/v2/AggregatingStateDescriptor.java:
##########
@@ -72,6 +72,23 @@ public AggregatingStateDescriptor(
this.aggregateFunction = checkNotNull(aggregateFunction);
}
+ /**
+ * Creates a new state descriptor with the given name, function, and type.
+ *
+ * <p>If this constructor fails (because it is not possible to describe
the type via a class),
Review Comment:
This is more like java specific. The most simple one is, you can't describe
`Pojo<A>` via a class since it has type parameter A. There are more complex
ones. I don't think Flink should detail this.
--
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]
