mhenc commented on code in PR #27892:
URL: https://github.com/apache/airflow/pull/27892#discussion_r1042348366
##########
airflow/config_templates/config.yml:
##########
@@ -418,6 +418,18 @@
type: string
default: ~
example: '{"some_param": "some_value"}'
+ - name: database_access_isolation
+ description: Whether components should use Airflow Internal API for DB
connectivity.
+ version_added: 2.6.0
+ type: boolean
+ example: ~
+ default: "False"
+ - name: database_api_url
Review Comment:
Renamed the configuration name
I think we shouldn't reinvent the wheel there - there are good LB solutions
that could be used in front of the multiple Airflow Internal API servers.
Unless there is a reason to use L7 LB (not sure about though)
--
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]
