:+1: for the current implementation. I would prefer that we not use update_columns. If callbacks are doing things they shouldn't they should be fixed or moved elsewhere.
Also :+1: because this should cause fewer unnecessary queries from the OrderUpdater.