turbaszek commented on a change in pull request #14492:
URL: https://github.com/apache/airflow/pull/14492#discussion_r584512624



##########
File path: airflow/providers/airbyte/operators/airbyte.py
##########
@@ -31,23 +31,36 @@ class AirbyteTriggerSyncOperator(BaseOperator):
     :type airbyte_conn_id: str
     :param connection_id: Required. The Airbyte ConnectionId UUID between a 
source and destination
     :type connection_id: str
+
     :param timeout: Optional. The amount of time, in seconds, to wait for the 
request to complete.
     :type timeout: float
     """
 
     @apply_defaults
     def __init__(
-        self, airbyte_conn_id: str, connection_id: str, timeout: Optional[int] 
= 3600, **kwargs
+        self,
+        airbyte_conn_id: str,

Review comment:
       ```suggestion
           airbyte_conn_id: str = "default_airbyte_conn",
   ```
   
   Most of operators and hooks usually have default conn_id




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


Reply via email to