eladkal opened a new issue #9099: URL: https://github.com/apache/airflow/issues/9099
with the addition of BranchSqlOperator https://github.com/apache/airflow/pull/8942 and as pointed during the review I'd like to suggest the following : ``` airflow/operators/sql_branch_operator.py airflow/operators/check_operator.py ``` will be merged to `airflow/operators/sql.py ` The operators will be renamed as: CheckOperator -> SqlCheckOperator ValueCheckOperator -> SqlValueCheckOperator IntervalCheckOperator -> SqlIntervalCheckOperator ThresholdCheckOperator ->SqlThresholdCheckOperator BranchSqlOperator -> the same. The motivation is that Airflow currently have two separated files in core related to generic SQL. My suggestion is similar to how PythonOperator(s) are organized in `python.py` (PythonOperator, BranchPythonOperator, ...) @samuelkhtu you mentioned on slack that you want to take it. If not I'll add PR for it ---------------------------------------------------------------- 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]
