chronitis edited a comment on issue #5231: [AIRFLOW-4456] Add BranchOperator URL: https://github.com/apache/airflow/pull/5231#issuecomment-489094949 Here's a possible refactoring; moving the logic part into `SkipMixin` and building both `BranchOperator` and `BranchPythonOperator` off it. It's possible that `BranchOperator` is unnecessary now and users should just inherit `SkipMixin` directly in this logic. (Tests pass except for the Kubernetes case which appears to have timed out).
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
