[GitHub] [superset] ktmud commented on pull request #17360: fix: Change datatype of column type in BaseColumn to allow larger datatype names for complexed columns

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

ktmud commented on pull request #17360:
URL: https://github.com/apache/superset/pull/17360#issuecomment-962252289


   I'm wondering if we can have two columns for this... store 95% cases in VACHAR with a reasonable limit and use TEXT to store large ENUMs and more advanced structs. Then there can be some helper functions to translate the complex types to more generic types to be used by the UI.


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