WencongLiu commented on code in PR #24398:
URL: https://github.com/apache/flink/pull/24398#discussion_r1510727272
##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/datastream/KeyedStream.java:
##########
@@ -1071,6 +1077,110 @@ protected SingleOutputStreamOperator<T>
aggregate(AggregationFunction<T> aggrega
return reduce(aggregate).name("Keyed Aggregation");
}
+ @Override
+ public PartitionWindowedStream<T> fullWindowPartition() {
+ throw new UnsupportedOperationException(
+ "KeyedStream doesn't support processing non-keyed
partitions.");
Review Comment:
After offline discussion, we all agree that a better version is "KeyedStream
doesn't support full window on partitions, if you want operations over
each-key, use methods in KeyedStream directly." In DataStream V1, the
KeyedStream doesn't have the concept "all records of same key is a partition".
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
