ashb commented on a change in pull request #6610: [AIRFLOW-6016] Remove
redundant setting `remote_logging`
URL: https://github.com/apache/airflow/pull/6610#discussion_r349701261
##########
File path: docs/howto/write-logs.rst
##########
@@ -157,9 +154,10 @@ First, to use the handler, ``airflow.cfg`` must be
configured as follows:
[core]
# Airflow can store logs remotely in AWS S3, Google Cloud Storage or
Elastic Search.
# Users must supply an Airflow connection id that provides access to the
storage
- # location. If remote_logging is set to true, see UPDATING.md for
additional
- # configuration requirements.
- remote_logging = True
+ # location. To set up correctly, see
https://airflow.readthedocs.io/en/stable/howto/write-logs.html
Review comment:
General point: Don't link to readthedocs.io but to airflow.apache.org
----------------------------------------------------------------
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
