>> > + operations = [ >> > + migrations.AddField( >> > + model_name='patchcomment', >> > + name='addressed', >> > + field=models.BooleanField(default=False), > > Thinking out loud: I suspect this will be an expensive migration since it will > add a new non-nullable field to all patchcomment rows. Maybe that's something > we > can live with, but making this nullable might make more sense, particularly if > we want to make this feature enableable/disableable on a project-by-project > basis (I don't know if we do).
I tested this on MariaDB 10.4 and it was ~instantaneous even with my large data set. I also tested it with Postgres and it was also less than a second. I think we're fine as is. Kind regards, Daniel _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork
