Skip to content

Commit

Permalink
Remove redundant logging in SFTP Hook (#16704)
Browse files Browse the repository at this point in the history
There is no equivalent logging in the store method – and arguably, a user of this hook who
wants this sort of operation logging would want more information here such as bytes transferred,
transfer rate, etc.

GitOrigin-RevId: 81be82bfb73f263ecd3b2d5f664e9c1ea751408a
  • Loading branch information
malthe authored and Cloud Composer Team committed Mar 10, 2022
1 parent 7ab6c96 commit 566e084
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions airflow/providers/sftp/hooks/sftp.py
Original file line number Diff line number Diff line change
Expand Up @@ -224,9 +224,7 @@ def retrieve_file(self, remote_full_path: str, local_full_path: str) -> None:
:type local_full_path: str
"""
conn = self.get_conn()
self.log.info('Retrieving file from FTP: %s', remote_full_path)
conn.get(remote_full_path, local_full_path)
self.log.info('Finished retrieving file from FTP: %s', remote_full_path)

def store_file(self, remote_full_path: str, local_full_path: str) -> None:
"""
Expand Down

0 comments on commit 566e084

Please sign in to comment.