mistercrunch commented on pull request #11714: URL: https://github.com/apache/incubator-superset/pull/11714#issuecomment-738450215
@ktmud I agree about the `path_no_param` naming & approach comment, but I think identifying params may require a bit more instrumentation and work. About reverting (if that's on the table), we kind of need to alter the upgrade to be upgrade script to be noop AND add a downgrade script that would remove the columns if they exist as people are already on this migration. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
