ktmud commented on a change in pull request #11256:
URL: 
https://github.com/apache/incubator-superset/pull/11256#discussion_r504528172



##########
File path: UPDATING.md
##########
@@ -25,6 +25,8 @@ assists people when migrating to a new version.
 
 * [11244](https://github.com/apache/incubator-superset/pull/11244): The 
`REDUCE_DASHBOARD_BOOTSTRAP_PAYLOAD` feature flag has been removed after being 
set to True for multiple months.
 
+* [11098](https://github.com/apache/incubator-superset/pull/11098): includes a 
database migration that adds a column caled `uuid` to all models, and updates 
the `Dashboard.position_json` column to also have UUIDs of charts. Depending on 
the number of objects the migration might take up to 3-4 minutes, requiring 
planning for the downtime.

Review comment:
       nit: I'm no native speaker, but the two "the"'s in the last sentence 
(the number of objects, the downtime) seems not necessary?




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



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

Reply via email to