You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/05 14:09:08 UTC

[GitHub] [airflow] kaxil commented on a change in pull request #14581: Rename last_scheduler_run into last_parsed_time, and ensure it's updated in DB

kaxil commented on a change in pull request #14581:
URL: https://github.com/apache/airflow/pull/14581#discussion_r588321242



##########
File path: airflow/migrations/versions/2c6edca13270_resource_based_permissions.py
##########
@@ -30,7 +30,7 @@
 
 # revision identifiers, used by Alembic.
 revision = '2c6edca13270'
-down_revision = '849da589634d'
+down_revision = '2e42bb497a22'

Review comment:
       this will create issues for customers updating from 2.0.1 since this migration already exist in 2.0.0 and 2.0.1
   
   why do we need this change? I read https://github.com/apache/airflow/pull/14581#issuecomment-791352402 but wasn't able to fully understand




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