[GitHub] [pinot] tibrewalpratik17 commented on pull request #10873: Use pinot-query-planner utils to extract table names in pinot-controller

["tibrewalpratik17 (via GitHub)" <gi...@apache.org>]

tibrewalpratik17 commented on PR #10873:
URL: https://github.com/apache/pinot/pull/10873#issuecomment-1607232924

   hey @walterddr i was looking into the integration test failure and this is only happening for queries with `BETWEEN` clause. 
   We lose the table information from RelNode at this method `trimUnusedFields` of Calcite - https://github.com/apache/pinot/blob/6a47311c03fdc020a4e05a3acf5cacb7843e3307/pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java#L292
   
   Digging deeper into the Calcite method shows that the `LogicalTableScan` node gets converted to `LogicalValues` node in this particular scenario. Testing this locally with `BETWEEN` clause on quickstart table works fine for me.
   
   Is this a known issue?


-- 
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-unsubscribe@pinot.apache.org

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


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