twalthr commented on a change in pull request #7177: [FLINK-7599] [table]
Support for aggregates in MATCH_RECOGNIZE
URL: https://github.com/apache/flink/pull/7177#discussion_r237439450
##########
File path:
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/match/MatchOperatorValidationTest.scala
##########
@@ -145,22 +139,21 @@ class MatchOperatorValidationTest extends TableTestBase {
|MATCH_RECOGNIZE (
| ORDER BY proctime
| MEASURES
- | A.symbol AS aSymbol
- | ALL ROWS PER MATCH
+ | SUM(A.price + B.tax) AS taxedPrice
| PATTERN (A B)
| DEFINE
- | A AS symbol = 'a'
+ | A AS A.symbol = 'a'
|) AS T
|""".stripMargin
streamUtils.tableEnv.sqlQuery(sqlQuery).toAppendStream[Row]
}
@Test
- def testGreedyQuantifierAtTheEndIsNotSupported(): Unit = {
- thrown.expectMessage("Greedy quantifiers are not allowed as the last
element of a " +
- "Pattern yet. Finish your pattern with either a simple variable or
reluctant quantifier.")
- thrown.expect(classOf[TableException])
+ def testAggregatesOnMultiplePatternVariablesNotSupportedInUDAGs(): Unit = {
+ thrown.expect(classOf[ValidationException])
Review comment:
Add part of the message here as well?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
