dongxiaoman commented on PR #8713: URL: https://github.com/apache/pinot/pull/8713#issuecomment-1130248790
> > @Jackie-Jiang Agree that using dot in table name is not good practice. I already saw JDBC H2 error if I copy/paste the Full integration test because JDBC engine treat dots as separator by default. That is why I hide the support with a feature flag. cc @apucher Meanwhile, as I mentioned, table name with dots can provide a transition/middle ground before we fully support full multi-tenant/DB concept. By this PR, we can allow the user write something naturally: `SELECT col1 from mydb.test_table where xxx` while the `mydb.test_table` is managed by Pinot admins, and the user is assigned `mydb` from the DataWarehouse infrastructure on top of Pinot. If this is not breaking other features, providing extra flexibility (extra knob) will help support more use cases, it actually expands Pinot's scope -- 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 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
