superset-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] mistercrunch opened a new pull request #5121: [sql lab] Fix issue around VARBINARY type in Presto
Date Fri, 01 Jun 2018 04:57:42 GMT
mistercrunch opened a new pull request #5121: [sql lab] Fix issue around VARBINARY type in
Presto
URL: https://github.com/apache/incubator-superset/pull/5121
 
 
   When receiving a VARBINARY field out of Presto, it shows up as type
   `bytes` out of the pyhive driver. Then the pre 3.15 version of
   simplejson attempts to convert it to utf8 by default and it craps out.
   
   I bumped to simplejson>=3.25.0 and set `encoding=None` as documented
   here
   https://simplejson.readthedocs.io/en/latest/#basic-usage so that we can
   handle bytes on our own.

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

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


Mime
View raw message