Re: [PR] IGNITE-21920 Cover SQL E051-04 (Basic query specification, GROUP BY can contain columns not in ) feature by tests [ignite-3] via GitHub Tue, 09 Apr 2024 02:20:31 -0700 ygerzhedovich commented on code in PR #3572: URL: https://github.com/apache/ignite-3/pull/3572#discussion_r1557294340 ########## modules/sql-engine/src/integrationTest/sql/aggregate/group/test_group_by_not_in_select.test: ########## Review Comment: let's add test with HAVING -- 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: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Previous message View by thread View by date Next message [PR] IGNITE-21920 Cover SQL E051-04 (Basic query specification,... via GitHub Re: [PR] IGNITE-21920 Cover SQL E051-04 (Basic query speci... via GitHub Re: [PR] IGNITE-21920 Cover SQL E051-04 (Basic query speci... via GitHub Re: [PR] IGNITE-21920 Cover SQL E051-04 (Basic query speci... via GitHub Reply via email to

[via GitHub]

ygerzhedovich commented on code in PR #3572:
URL: https://github.com/apache/ignite-3/pull/3572#discussion_r1557294340


##########
modules/sql-engine/src/integrationTest/sql/aggregate/group/test_group_by_not_in_select.test:
##########


Review Comment:
   let's add test with HAVING



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org