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:
[email protected]


With regards,
Apache Git Services

Reply via email to