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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
