You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@superset.apache.org by GitBox <gi...@apache.org> on 2018/02/02 11:41:19 UTC

[GitHub] xrmx commented on issue #4329: Druid v0.6 compatibility (second try)

xrmx commented on issue #4329: Druid v0.6 compatibility (second try)
URL: https://github.com/apache/incubator-superset/pull/4329#issuecomment-362564316
 
 
   Just a couple of thoughts:
   - any chance that pydruid sqlalchemy dialect already handles that?
   - passing the version as parameter on an object does not look that nice, either storing the version as an attribute in the object or creating a subclass for older apis may look better

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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