[GitHub] [airflow] potiuk commented on issue #5199: [AIRFLOW-4401] SynchronizedQueue used where empty() is used.

2019-04-29 Thread GitBox
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.

[GitHub] [airflow] potiuk commented on issue #5199: [AIRFLOW-4401] SynchronizedQueue used where empty() is used.

2019-04-28 Thread GitBox
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