You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2019/05/01 17:06:36 UTC
[GitHub] [incubator-superset] villebro commented on issue #6610: Add support
to Apache Drill
villebro commented on issue #6610: Add support to Apache Drill
URL: https://github.com/apache/incubator-superset/pull/6610#issuecomment-488344284
@cgivre as this PR in its current state would break support for existing engines I don't think it can be merged. I would propose putting through a PR first that adds support for period separated table names, and once that's merged, rebasing this and merging once the dust settles. I have an idea of how this could be done with minimal collateral damage. @mistercrunch if this sounds ok I can put through a PR that makes minimal changes required to add support for period separated table names.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org