pankajkoti commented on code in PR #39275: URL: https://github.com/apache/airflow/pull/39275#discussion_r1581018600
########## docs/apache-airflow/migrations-ref.rst: ########## @@ -39,7 +39,10 @@ Here's the list of all the Database Migrations that are executed via when you ru +---------------------------------+-------------------+-------------------+--------------------------------------------------------------+ | Revision ID | Revises ID | Airflow Version | Description | +=================================+===================+===================+==============================================================+ -| ``677fdbb7fc54`` (head) | ``1949afb29106`` | ``2.10.0`` | add new executor field to db | +| ``bff083ad727d`` (head) | ``677fdbb7fc54`` | ``2.9.1`` | Remove ``idx_last_scheduling_decision`` index on | +| | | | last_scheduling_decision in dag_run table | ++---------------------------------+-------------------+-------------------+--------------------------------------------------------------+ +| ``677fdbb7fc54`` | ``1949afb29106`` | ``2.10.0`` | add new executor field to db | Review Comment: My first migration PR here. I am wondering if we need to do some work around for the head here. Meaning head is pointing to 2.9.1, but we already added an index before for 2.10. If someone knows we need to handle this differently would be helpful. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
