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

   > Would it be possible to split this PR? This is really a "beast" to review.
   
   Unfortunately ... I am afraid it's not easy. I thpught about it, but I was 
not able to figure out how to split it. 
   There are many connected things and a bit of "chain reaction" -> removing of 
providers tests requires to move imports, which (for example cc: @vincbeck ) 
make us move all the remaining "core" tests that accessed FAB permission model 
to be moved to fab, it also makes it necessary to move integration tests there 
etc., etc. ...
   
   So - as much I would love to, I am afraid it willl be very difficult if not 
impossible.
   
   The good thing about it is that there are many removals and similar changes 
across the board.
   
   I am really sorry for the reviewers, but I am afraid we have to bite the 
bullet.
   
   
   BTW. It is finally GREEN....


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