Taragolis commented on code in PR #33786:
URL: https://github.com/apache/airflow/pull/33786#discussion_r1320349216


##########
airflow/providers/qubole/operators/qubole.py:
##########


Review Comment:
   Qubole is suspended provider.



##########
airflow/providers/amazon/aws/operators/emr.py:
##########
@@ -1011,6 +1011,7 @@ def __init__(
         deferrable: bool = conf.getboolean("operators", "default_deferrable", 
fallback=False),
         **kwargs,
     ):
+        # FIXME: Handle if statement(s) in execute() method to comply with 
validate-operators-init pre-commit
         if waiter_check_interval_seconds is NOTSET:

Review Comment:
   I'm sure that this not required fixed,`NOTSET` it is special sentinel which 
should not provided by user manually.



-- 
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]

Reply via email to