uranusjr commented on a change in pull request #16110:
URL: https://github.com/apache/airflow/pull/16110#discussion_r651709940
##########
File path: airflow/__main__.py
##########
@@ -34,6 +35,15 @@ def main():
os.environ['KRB5CCNAME'] = conf.get('kerberos', 'ccache')
os.environ['KRB5_KTNAME'] = conf.get('kerberos', 'keytab')
+ # if dags folder has to be set to configured value, make sure it is set
properly (needed on Dask-Workers)
Review comment:
Windows API can handle both forward and backward slashes (except some
very old legacy stuff, none of which is used by Python’s path libraries), so it
should be fine if we use forward slashes everywhere. For rare cases where some
third-party packages rely on legacy APIs, we can call `os.path.normpath` to
convert the slashes.
--
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]
