uranusjr edited a comment on pull request #19022:
URL: https://github.com/apache/airflow/pull/19022#issuecomment-950571419


   Another way to do this would be to simply use query parameters like how we 
handle database URLs. The user can set `REMOTE_BASE_LOG_FOLDER` to (say)
   
   ```
   
cloudwatch://my.cloudwatch.server/?create_log_group=false&create_log_stream=false
   ```
   
   Which is (fortunately) possible because `log_group_arn` only uses the netloc 
part. But I think using standalone configurations (like your snippet above) is 
more idiomatic to Airflow, similar to configs for Elastic etc.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to