[GitHub] [incubator-superset] xinbinhuang commented on issue #8874: [SIP-33] Proposal for Removing SQLite Support for Metadata Databases

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

xinbinhuang commented on issue #8874: [SIP-33] Proposal for Removing SQLite Support for Metadata Databases
URL: https://github.com/apache/incubator-superset/issues/8874#issuecomment-580051961
 
 
   Hi @willbarrett , 
   
   I am pretty new to superset so my comment may not be accurate, but regarding the batch migrations:
   
   > SQLite support causes multiple pain points for developers, including bad migration practices with unnecessary batch of migrations 
   
   This should not have an actual impact on the migration process. The method `bathch_alter_table` only does an actual batch migration on SQLite, while keeping the normal `ALTER` operations on other databases. [Alembic batch](https://alembic.sqlalchemy.org/en/latest/batch.html)
   
   As for SQLite for production, I am neither support nor against it, as it does give you an easy deployment with small users' size. However, as we are living in a world of containerization, deployment is not an issue either. 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org