harsh481 opened a new pull request, #55569: URL: https://github.com/apache/airflow/pull/55569
## Add configurable confirm parameter to S3ToSFTPOperator ### Summary This enhancement adds a `confirm` parameter to the S3ToSFTPOperator to make SFTP file transfer confirmation behavior explicit and configurable, providing consistency with SFTPOperator. ### Changes - Add `confirm: bool = True` parameter to S3ToSFTPOperator constructor - Pass confirm parameter explicitly to Paramiko's `put()` method - Update docstring with parameter documentation - Add comprehensive unit tests for both `confirm=True` and `confirm=False` scenarios ### Motivation While Paramiko defaults to `confirm=True`, this change allows users to: - Explicitly control confirmation behavior - Choose `confirm=False` for performance-critical scenarios - Have consistent API design with SFTPOperator ### Testing - Added unit tests verifying both confirmation scenarios - All existing tests continue to pass - Code passes static analysis checks ### Backward Compatibility This change is fully backward compatible. Existing code will continue to work exactly as before since the default value `confirm=True` matches Paramiko's existing default behavior. -- 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]
