superset-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-superset] cgivre commented on issue #6610: Add support to Apache Drill
Date Thu, 02 May 2019 06:18:38 GMT
cgivre commented on issue #6610: Add support to Apache Drill
URL: https://github.com/apache/incubator-superset/pull/6610#issuecomment-488563850
 
 
   I would be an advocate of the second approach that @mistercrunch mentioned where we have
a better data structure.  However, I'm not sure I know enough about the internals of Superset
to implement those kinds of changes.  (Unless I'm wrong and that would be an easy fix)
   
   It also does seem out of scope for this PR.  Should we create a separate PR for that specific
issue? @villebro could you assist with the implementation?  

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


Mime
View raw message