[
https://issues.apache.org/jira/browse/FLINK-5219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15865637#comment-15865637
]
ASF GitHub Bot commented on FLINK-5219:
---------------------------------------
Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/3266#discussion_r101014900
--- Diff:
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/dataset/DataSetWindowAggregate.scala
---
@@ -281,9 +279,36 @@ class DataSetWindowAggregate(
.returns(rowTypeInfo)
.name(aggregateOperatorName)
.asInstanceOf[DataSet[Any]]
+
+ } else {
+ // non-grouping window
+ val preMapPartitionFunction =
createDataSetWindowAggregationMapPartitionFunction(
+ window,
+ namedAggregates,
+ inputType)
+
+ val mapPartitionFunction =
createDataSetWindowAggregationMapPartitionFunction(
+ window,
+ namedAggregates,
+ inputType,
+ rowRelDataType,
+ namedProperties,
+ isPreMapPartition = false,
+ isInputCombined = true)
+
+ mappedInput.sortPartition(rowTimeFieldPos, Order.ASCENDING)
+ .mapPartition(preMapPartitionFunction)
+ .sortPartition(windowStartPos, Order.ASCENDING).setParallelism(1)
+ .sortPartition(windowEndPos, Order.ASCENDING).setParallelism(1)
+ .mapPartition(mapPartitionFunction).setParallelism(1)
--- End diff --
I think we can also use `.reduceGroup()` and a `GroupReduceFunction` here.
Without `groupBy`, the `GroupReduceFunction` will be executed with parallelism
1.
> Add non-grouped session windows for batch tables
> ------------------------------------------------
>
> Key: FLINK-5219
> URL: https://issues.apache.org/jira/browse/FLINK-5219
> Project: Flink
> Issue Type: Sub-task
> Components: Table API & SQL
> Reporter: sunjincheng
> Assignee: sunjincheng
>
> Add non-grouped session windows for batch tables as described in
> [FLIP-11|https://cwiki.apache.org/confluence/display/FLINK/FLIP-11%3A+Table+API+Stream+Aggregations].
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
