feng-tao commented on issue #4305: [AIRFLOW-3397] Fix integrety error in rbac 
AirflowSecurityManager
URL: 
https://github.com/apache/incubator-airflow/pull/4305#issuecomment-446301765
 
 
   thanks @BrechtDeVlieger for the fix. It makes sense. With your change I 
assume the integrety  error is fixed? Not sure why I didn't catch that 
initially.  @ashb , I believe the existing unit test should cover this case 
from the functional perspective. And I believe the issue only happens if there 
are multiple gunicorn processes started which is very difficult to do unit test.
   
   I prefer to merge this as it fixes the regression. Let me know what you 
think.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to