xinbinhuang commented on a change in pull request #15680:
URL: https://github.com/apache/airflow/pull/15680#discussion_r627077699



##########
File path: airflow/providers/amazon/aws/transfers/mongo_to_s3.py
##########
@@ -117,7 +117,6 @@ def execute(self, context) -> bool:
                 mongo_collection=self.mongo_collection,
                 query=cast(dict, self.mongo_query),
                 mongo_db=self.mongo_db,
-                allowDiskUse=self.allow_disk_use,

Review comment:
       @amatellanes  I'm ok with just removes it for now, and we can think 
about how to add it back in another PR in the future when someone requests it. 
Or we can handle the branching logic in this PR so that _allow_disk_use_ can 
play with single query as well.
   
   If you wanna go with the former, please update the docstring to reflect the 
fact that _allow_disk_use_ doesn't apply to single query (`.find`).




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