[GitHub] [superset] hughhhh commented on pull request #19701: fix: convert column type to str during dual read/write

[GitBox <gi...@apache.org>]

hughhhh commented on PR #19701:
URL: https://github.com/apache/superset/pull/19701#issuecomment-1098532908

   > Should we save the original database type here? The type field specifically used Text as storage so the originally intention is probably to allow storing the full schema of complex types such as ARRAY and STRUCT?
   > 
   > What does the old column model save?
   
   I'm pretty sure it was still ARRAY, but this is only a patch i'm pretty sure there other types we'd have to map to strings later


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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