sungpeo commented on a change in pull request #21538:
URL: https://github.com/apache/airflow/pull/21538#discussion_r807393371



##########
File path: UPDATING.md
##########
@@ -80,6 +80,13 @@ https://developers.google.com/style/inclusive-documentation
 
 -->
 
+### You can't name zip files as ending with .py for DAGs
+
+It was previously possible to load any files as zip files in the DAGs folder 
via `zipfile.is_zipfile`.
+
+Now .py (extension) files are going to be loaded as modules without checking 
whether it is zipfile. (Less IO)
+If .py file in the DAGs folder were a zip compressed file, processing it would 
be failed with exception.

Review comment:
       Thanks fot that. Pushed the commit suggestion.




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