jscheffl commented on PR #59739:
URL: https://github.com/apache/airflow/pull/59739#issuecomment-3686835935

   Same comment applies to this PR like in #59741:
   
   Thanks for the fix and looking good - except a small "pitfall" that might 
arise - you are altering the 3.0.0 migration means everybody migrating AFTER 
this PR landed in a release if migrating from before 3.0.0 will get this 
applied. If I earlier had migrated from 2.11.0 to 3.1.5 and then (later) will 
upgrade to 3.2.0 (assume this PR then is on main) will miss the `NOT NULL` 
change in the DB layout.


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

Reply via email to