[GitHub] [pinot] walterddr commented on issue #10701: TableName extraction fails for tables with name pattern as `db.tablename` in CalciteSqlParser

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

walterddr commented on issue #10701:
URL: https://github.com/apache/pinot/issues/10701#issuecomment-1529124114

   this is actually not suppose to work. to a complete SQL system `db.table1` means `{"catalog": "db", "table": "table1"}` 
   however pinot doesn't support catalog thus any prepended catalog is resolved as "default" thus the integration test case is to verify that for the existing query engine. 
   
   However for the new engine, we actually implicitly added the default catalog name since Calcite's catalog reader requires a root schema (or root catalog if you will) thus it will not be able to find a `db` catalog internally. 
   
   We are yet to discuss the catalog support going forward but as far as for now this will not be supported as "ignoring the catalog and only parse table name, then move on with what we had" is not really something we should do


-- 
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