You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/02/02 06:13:42 UTC

[GitHub] [airflow] yuqian90 commented on issue #7276: [AIRFLOW-5391] Make BaseOperator respect branching of parent BranchPythonOperator

yuqian90 commented on issue #7276: [AIRFLOW-5391] Make BaseOperator respect branching of parent BranchPythonOperator
URL: https://github.com/apache/airflow/pull/7276#issuecomment-581103041
 
 
   - Added tests
   - Extended the same support to ``BranchPythonOperator``, ``ShortCircuitOperator`` and ``BaseBranchOperator``. Clearing downstream tasks of all these three operators will now do the right thing.
   - That said, I still think ``BaseBranchOperator`` is a bit redundant since we already have ``BranchPythonOperator``. I'm proposing to deprecate ``BaesBranchOperator`` in [another PR](https://github.com/apache/airflow/pull/7331)
   
   The PR is now ready for review. Reviewers @kaxil  @ashb  @feluelle please take a look.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services