pingzh commented on PR #28367: URL: https://github.com/apache/airflow/pull/28367#issuecomment-1353698604
> It may not be a good idea to reuse `mkdirs`. Its docstring specifically says _If directory already exists, this is a no-op_ and the new implementation is not compatible with the previous behaviour. @uranusjr i think the new implementation is no-op when the dir already exists. it is actually from the previous implementation, e.g. 1.10.15 https://github.com/apache/airflow/blob/5786dcdc392f7a2649f398353a0beebef01c428e/airflow/utils/file.py#L55-L73 -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org