@feng-tao thanks for reviewing on this, I would need some suggestions here: 1. Since this is refactoring changes, the tests/core.py should cover the unit tests for them, for example, https://github.com/apache/incubator-airflow/blob/master/tests/core.py#L1493. Should I add more unit test? 2. I can remove pool class from api client, but my question is should we continue using `airflow.api.common.experimental` for pool or have all methods in Pool class like we did for `Variable`?
[ Full content available at: https://github.com/apache/incubator-airflow/pull/3858 ] This message was relayed via gitbox.apache.org for [email protected]
