ktmud commented on PR #20385: URL: https://github.com/apache/superset/pull/20385#issuecomment-1155865232
> I'm also wondering whether it's worth creating a new type like [MediumText](https://github.com/apache/superset/blob/32bb1ce3ff3ae9b7e54708463f152326d35ceb98/superset/utils/core.py#L1512-L1513) and update key_value/models to point to this type as well. It's risky when the declarative model is out of sync with the actual db schema. Highlighting [this comment](https://github.com/apache/superset/pull/19805#issuecomment-1105416580) since it obviously got lost in the original PR. -- 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]
