[GitHub] [airflow] potiuk commented on pull request #27629: Bump ```alembic``` minimum version

2022-11-13 Thread GitBox


potiuk commented on PR #27629:
URL: https://github.com/apache/airflow/pull/27629#issuecomment-1312965089

   Thanks @aa3pankaj for figuring that one out :). 


-- 
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: commits-unsubscr...@airflow.apache.org

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



[GitHub] [airflow] potiuk commented on pull request #27629: Bump ```alembic``` minimum version

2022-11-13 Thread GitBox


potiuk commented on PR #27629:
URL: https://github.com/apache/airflow/pull/27629#issuecomment-1312964879

   > It’s likely not worthwhile to debug to deep into this, so I think this is 
good enough. We can always review and revise the version range if issues come 
up.
   
   Correct - we have a number of "lower bounds" and unlike "upper bounds" - we 
have no  to explain the lower bounds reasons in detail or evan dig-deeply into 
the reasons, as long as the "lower-bound" version is "old enough". It might be 
interesting to know but it has mostly academic value to know it as with every 
day the reason for that lower-bound is farhter in the past and becomes less and 
less relevant(as opposed to upper-bound - every day we keep it, it becomes more 
and more relevant and annoying). 
   


-- 
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: commits-unsubscr...@airflow.apache.org

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