rusackas commented on PR #42203:
URL: https://github.com/apache/superset/pull/42203#issuecomment-5107852327

   Nice bit of detective work tracing this from the wire-protocol FIELD_TYPE 
names down to the tuple/list fetchall bug, both fixes look right and the tests 
cover them well. Did you try this against a real BLOB column with actual binary 
data, or just TEXT-ish content? Curious if STRING is the right generic type 
there or if it'll choke on non-utf8 bytes.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to