betodealmeida commented on pull request #13216: URL: https://github.com/apache/superset/pull/13216#issuecomment-781593465
> I think another way to do this is do the column check in `add_uuid_column_to_import_mixin.py`, then simply re-run the same `upgrade` again in `add_missing_uuid_column.py`. This way you avoid the maximum amount of duplicate code, but I guess current approach works, too. The problem of modifying the original script is that it forces people to downgrade and upgrade again to fix their database, while adding a new script they just need to upgrade. ---------------------------------------------------------------- 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]
