[GitHub] [superset] hughhhh commented on pull request #19767: fix: Fix migration for removing time_range_endpoints 3

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

hughhhh commented on PR #19767:
URL: https://github.com/apache/superset/pull/19767#issuecomment-1102231382

   Verifed locally that this migration works:
   ```
   >> select count(*) from slices where query_context like '%time_range_endpoints%'
   5
   
   >> superset db upgrade
   INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
   INFO  [alembic.runtime.migration] Will assume transactional DDL.
   INFO  [alembic.runtime.migration] Running upgrade cecc6bf46990 -> ad07e4fdbaba, rm_time_range_endpoints_from_qc_3
   slices updated with no time_range_endpoints: 5
   
   >> select count(*) from slices where query_context like '%time_range_endpoints%'
   0
   ```


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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