walterddr opened a new issue, #10688:
URL: https://github.com/apache/pinot/issues/10688

   see:
   ```
   09:57:38.796 ERROR [Connection] [main] Cannot parse table name from query: 
select D_YEAR, S_NATION, P_CATEGORY, sum(LO_REVENUE - LO_SUPPLYCOST) as profit
   from lineorder, dates, customer, supplier, part where LO_CUSTKEY = C_CUSTKEY
   and LO_SUPPKEY = S_SUPPKEY and LO_PARTKEY = P_PARTKEY and LO_ORDERDATE = 
D_DATEKEY
   and C_REGION = 'AMERICA' and S_REGION = 'AMERICA' and (D_YEAR = 1997 or 
D_YEAR = 1998)
   and (P_MFGR = 'MFGR#1' or P_MFGR = 'MFGR#2') group by D_YEAR, S_NATION, 
P_CATEGORY
   order by D_YEAR, S_NATION, P_CATEGORY;
   
   java.lang.IllegalStateException: Unsupported join type: COMMA
        at 
org.apache.pinot.sql.parsers.CalciteSqlParser.compileToJoin(CalciteSqlParser.java:552)
 ~[classes/:?]
        at 
org.apache.pinot.sql.parsers.CalciteSqlParser.compileToDataSource(CalciteSqlParser.java:528)
 ~[classes/:?]
        at 
org.apache.pinot.sql.parsers.CalciteSqlParser.compileSqlNodeToPinotQuery(CalciteSqlParser.java:475)
 ~[classes/:?]
        at 
org.apache.pinot.sql.parsers.CalciteSqlParser.compileToPinotQuery(CalciteSqlParser.java:244)
 ~[classes/:?]
        at 
org.apache.pinot.sql.parsers.CalciteSqlParser.compileToPinotQuery(CalciteSqlParser.java:239)
 ~[classes/:?]
        at 
org.apache.pinot.sql.parsers.CalciteSqlCompiler.compileToBrokerRequest(CalciteSqlCompiler.java:32)
 ~[classes/:?]
        at 
org.apache.pinot.client.Connection.resolveTableName(Connection.java:189) 
[classes/:?]
        at org.apache.pinot.client.Connection.execute(Connection.java:119) 
[classes/:?]
        at org.apache.pinot.client.Connection.execute(Connection.java:94) 
[classes/:?]
        at 
org.apache.pinot.integration.tests.SSBQueryIntegrationTest.testQueriesValidateAgainstH2(SSBQueryIntegrationTest.java:112)
 [test-classes/:?]
        at 
org.apache.pinot.integration.tests.SSBQueryIntegrationTest.testSSBQueries(SSBQueryIntegrationTest.java:106)
 [test-classes/:?]
        at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
~[?:?]
        at 
jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 ~[?:?]
        at 
jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 ~[?:?]
        at java.lang.reflect.Method.invoke(Method.java:566) ~[?:?]
        at 
org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:108)
 [testng-6.11.jar:?]
        at org.testng.internal.Invoker.invokeMethod(Invoker.java:661) 
[testng-6.11.jar:?]
        at org.testng.internal.Invoker.invokeTestMethod(Invoker.java:869) 
[testng-6.11.jar:?]
        at org.testng.internal.Invoker.invokeTestMethods(Invoker.java:1193) 
[testng-6.11.jar:?]
        at 
org.testng.internal.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:126)
 [testng-6.11.jar:?]
        at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:109) 
[testng-6.11.jar:?]
        at org.testng.TestRunner.privateRun(TestRunner.java:744) 
[testng-6.11.jar:?]
        at org.testng.TestRunner.run(TestRunner.java:602) [testng-6.11.jar:?]
   ```
   
   this is b/c the Connection code path doesn't go through the v2 engine (which 
only calls `CalciteSqlParser` for leaf stages, not `CalciteSqlCompiler` for the 
overall query


-- 
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: commits-unsubscr...@pinot.apache.org.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to