tkalkirill commented on code in PR #5113:
URL: https://github.com/apache/calcite/pull/5113#discussion_r3619415369
##########
core/src/test/java/org/apache/calcite/test/enumerable/EnumerableMergeUnionTest.java:
##########
@@ -176,16 +244,14 @@ class EnumerableMergeUnionTest {
tester(false,
new HrSchemaBig(),
"select * from (select empid, name from emps where name like 'G%'
union select empid, name from emps where name like '%l') order by name limit 3")
- .explainContains("EnumerableLimit(fetch=[3])\n"
+ .explainContains("EnumerableLimitSort(sort0=[$1], dir0=[ASC],
fetch=[3])\n"
Review Comment:
I noticed an issue at `UNION DISTINCT` and decided to fix it right away. I
suppose that wasn't the right move, as it resulted in too many changes and is
confusing. I think I'll fix it later in a separate ticket.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
