[ https://issues.apache.org/jira/browse/CALCITE-5048?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17520410#comment-17520410 ]
Ruben Q L commented on CALCITE-5048: ------------------------------------ Personally, I would refrain from touching {{RelMdRowCount}} because IMO it is out of the scope of the current ticket. I would focus the PR on the "parameterized LIMIT/OFFSET" cases to be handled (RelMdMaxRowCount & RelMdMinRowCount). Your point about {{RelMdRowCount}} is a valid one and perhaps should be discussed, but I believe it should be done on a separate, dedicated Jira ticket (maybe linked to the current one). > Query with parameterized LIMIT and correlated sub-query throws AssertionError > "not a literal" > --------------------------------------------------------------------------------------------- > > Key: CALCITE-5048 > URL: https://issues.apache.org/jira/browse/CALCITE-5048 > Project: Calcite > Issue Type: Bug > Components: core > Affects Versions: 1.29.0 > Reporter: Corvin Kuebler > Assignee: Jiajun Xie > Priority: Major > Labels: pull-request-available > Time Spent: 1h > Remaining Estimate: 0h > > This query: > {code:java} > // Does not run > @Test void testDynamicParameterLimit() { > CalciteAssert.hr() > .query("SELECT * FROM \"hr\".\"emps\" AS a " + > "WHERE \"deptno\" = " + > "(SELECT MAX(\"deptno\") " + > "FROM \"hr\".\"emps\" AS b " + > "WHERE a.\"empid\" = b.\"empid\"" + > ") LIMIT ?") > .runs(); > }{code} > results in the following exception: > {code:java} > not a literal: ?0 > java.lang.AssertionError: not a literal: ?0 > at org.apache.calcite.rex.RexLiteral.findValue(RexLiteral.java:1208) > at org.apache.calcite.rex.RexLiteral.intValue(RexLiteral.java:1183) > at > org.apache.calcite.sql2rel.RelDecorrelator.decorrelateRel(RelDecorrelator.java:481) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at org.apache.calcite.util.ReflectUtil$2.invoke(ReflectUtil.java:531) > at > org.apache.calcite.sql2rel.RelDecorrelator.getInvoke(RelDecorrelator.java:711) > at > org.apache.calcite.sql2rel.RelDecorrelator.decorrelate(RelDecorrelator.java:306) > at > org.apache.calcite.sql2rel.RelDecorrelator.decorrelateQuery(RelDecorrelator.java:230) > at > org.apache.calcite.tools.Programs$DecorrelateProgram.run(Programs.java:361) > at > org.apache.calcite.tools.Programs$SequenceProgram.run(Programs.java:336) > at org.apache.calcite.prepare.Prepare.optimize(Prepare.java:177) > at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:312) > at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:220){code} -- This message was sent by Atlassian Jira (v8.20.1#820001)