[GitHub] [airflow] potiuk commented on issue #5199: [AIRFLOW-4401] SynchronizedQueue used where empty() is used.
potiuk commented on issue #5199: [AIRFLOW-4401] SynchronizedQueue used where empty() is used. URL: https://github.com/apache/airflow/pull/5199#issuecomment-487815867 Closing in favour of #5200 . Managed Queue uses more standard approach with less synchronisation code to maintain. 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [airflow] potiuk commented on issue #5199: [AIRFLOW-4401] SynchronizedQueue used where empty() is used.
potiuk commented on issue #5199: [AIRFLOW-4401] SynchronizedQueue used where empty() is used. URL: https://github.com/apache/airflow/pull/5199#issuecomment-487394715 Seems that https://github.com/apache/airflow/pull/5200/ solves it in a simpler way. Let's wait for the build to finish, but it looks good from local hosting. 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services