mistercrunch commented on PR #33720: URL: https://github.com/apache/superset/pull/33720#issuecomment-2968369815
This seems like a win to me, though seeing `test-mysql` fail in CI (assuming it's related to this change) shows that this change might have some upgrade implications. If there are indeed change management issues, it may require: - adding a line in 1UPDATING.md` about required checks/changes around the the future upgrades - [I'm hoping that's not the case but needs to be ruled-out] maybe this would require a database migration script to re-serialize/migrate the json that we have serialized in some/all databases (?) Related GPT thread -> https://chatgpt.com/share/684b5999-8d50-8010-ab53-d62e10188b50 -- 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]
