Github user fhueske commented on the issue:
https://github.com/apache/flink/pull/2961
merging
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user fhueske commented on the issue:
https://github.com/apache/flink/pull/2961
Hi @KurtYoung,
thanks for the update and the nice solution for the window aggregates.
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply
Github user KurtYoung commented on the issue:
https://github.com/apache/flink/pull/2961
Hi @fhueske , i have added the case of selecting from GroupWindowedTable.
And have leave some test cases for EventTimeWindow in batch mode, since it
supports using arbitrary field as the time
Github user KurtYoung commented on the issue:
https://github.com/apache/flink/pull/2961
rebased to latest master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and