[ https://issues.apache.org/jira/browse/FLINK-8325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16318495#comment-16318495 ]
ASF GitHub Bot commented on FLINK-8325: --------------------------------------- Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5241#discussion_r160420917 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/batch/sql/JoinITCase.scala --- @@ -254,7 +254,7 @@ class JoinITCase( val table = CollectionDataSets.getSmall3TupleDataSet(env).toTable(tEnv).as('a1, 'a2, 'a3) tEnv.registerTable("A", table) - val sqlQuery2 = "SELECT * FROM (SELECT count(*) FROM A) CROSS JOIN A" --- End diff -- The current result is correct. `CollectionDataSets.getSmall3TupleDataSet` has three rows. Hence `SELECT count(a1) FROM A` must be `3` and not `4`. The PR breaks the correctness of the test. > Add COUNT AGG support constant parameter, i.e. COUNT(*), COUNT(1) > ------------------------------------------------------------------ > > Key: FLINK-8325 > URL: https://issues.apache.org/jira/browse/FLINK-8325 > Project: Flink > Issue Type: Bug > Components: Table API & SQL > Affects Versions: 1.5.0 > Reporter: sunjincheng > Assignee: sunjincheng > > COUNT(1) with Group Window, always output 0. > e.g. > DATA: > {code} > val data = List( > (1L, 1, "Hi"), > (2L, 2, "Hello"), > (4L, 2, "Hello"), > (8L, 3, "Hello world"), > (16L, 3, "Hello world")) > {code} > SQL: > {code} > SELECT b, COUNT(1) FROM MyTable GROUP BY Hop(proctime, interval '0.001' > SECOND, interval '0.002' SECOND),b > {code} > OUTPUT: > {code} > 1,0,1, > 1,0,1, > 2,0,1, > 2,0,1, > 2,0,2, > 3,0,1, > 3,0,1 > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)