remem9527 commented on code in PR #24451:
URL: https://github.com/apache/airflow/pull/24451#discussion_r897944996


##########
airflow/providers/sftp/sensors/sftp.py:
##########
@@ -71,6 +71,8 @@ def poke(self, context: 'Context') -> bool:
                 self.actual_file_to_check = file_from_pattern
             else:
                 return False
+        else:
+            self.actual_file_to_check = self.path

Review Comment:
   Personally, I think it would be very cool if I could use sensors like this:
   ```
   sensor_task = SFTPSensor(task_id='task_id', sftp_conn_id='sftp_conn_id', 
path='data/folder/datafile_{{ds}}_part_?.[ct]sv')
   ```
   Also, this supporting should be implemented on some base class, so that all 
the file sensors could be supported.



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