You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/05/19 12:04:12 UTC

[GitHub] [airflow] BasPH commented on issue #5301: [AIRFLOW-4537] Remove the mkdir_p function in favour of native Python pathlib

BasPH commented on issue #5301: [AIRFLOW-4537] Remove the mkdir_p function in favour of native Python pathlib
URL: https://github.com/apache/airflow/pull/5301#issuecomment-493751183
 
 
   Good point - pathlib is a higher level API. It's not necessary to use but does provide a (IMO) much clearer API.
   
   I'm always wary of these "magic" functions such as `airflow.utils.file.mkdirs`, which seem to handle some sort of undocumented edge case, so I'm not sure why they exist, because the mkdirs operation can be handled nicely with pathlib. Shall we replace it in a different PR?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services