eladkal commented on code in PR #63210:
URL: https://github.com/apache/airflow/pull/63210#discussion_r3074133283
##########
providers/common/sql/src/airflow/providers/common/sql/operators/sql.py:
##########
@@ -714,6 +714,7 @@ class SQLTableCheckOperator(BaseSQLOperator):
:param conn_id: the connection ID used to connect to the database
:param database: name of database which overwrite the defined one in
connection
+ :param accept_none: If True, do not fail if the table is empty
Review Comment:
```suggestion
:param accept_none: If True, an empty table (row count=0) will not
trigger a failure.
```
I think it's more clear? I somewhat find the 2nd if confusing(?) but not a
native speaker so i might be wrong.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
