eladkal commented on issue #9099: URL: https://github.com/apache/airflow/issues/9099#issuecomment-638051733
> Should we change the class name from BranchSqlOperator to SQLBranchOperator or use the Python organization? (BranchSQLOperator?) It won't be consistent with how it's done in `python.py `: **Branch**PythonOperator PythonOperator PythonVirtualenvOperator In my opinion the goal here is to make the SQL be consistent with Airflow current convention. After this step is done you can raise the question of `BranchXOperator` or `XBranchOperator` to discussion as this effect operators from different modules so it's not just SQL local module issue. ---------------------------------------------------------------- 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]
