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

   I thin that might get synchronization with FAB more difficult. This module 
is extracted and modified FAB `security/sqla/manager.py`  - and every time we 
bump FAB we also modify this file if there are relevant modificatons. As bad as 
it is (I am entirely unhappy with the way we have it) - I think until we change 
the approach, we should avoid modifying this file to make applying the 
modification easier (it's basically vendored-in but in a very twisted way).. 
   
   


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