You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/05/14 19:04:24 UTC
[GitHub] [airflow] jedcunningham opened a new pull request #15861: Default `resultBackendConnection` to `metadataConnection`
jedcunningham opened a new pull request #15861:
URL: https://github.com/apache/airflow/pull/15861
Instead of requiring anyone using an external db and CeleryExecutor to
set their database details twice, let the default for
`resultBackendConnection` be the values from `metadataConnection`.
Anyone who wants to use a separate backend for results still can.
--
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:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #15861: Default `resultBackendConnection` to `metadataConnection`
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #15861:
URL: https://github.com/apache/airflow/pull/15861#issuecomment-841509987
[The Workflow run](https://github.com/apache/airflow/actions/runs/843305397) is cancelling this PR. Building images for the PR has failed. Follow the workflow link to check the reason.
--
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:
users@infra.apache.org
[GitHub] [airflow] kaxil commented on pull request #15861: Default `resultBackendConnection` to `metadataConnection`
Posted by GitBox <gi...@apache.org>.
kaxil commented on pull request #15861:
URL: https://github.com/apache/airflow/pull/15861#issuecomment-841526889
Ran the tests locally -- worked fine, merging
--
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:
users@infra.apache.org
[GitHub] [airflow] kaxil merged pull request #15861: Default `resultBackendConnection` to `metadataConnection`
Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #15861:
URL: https://github.com/apache/airflow/pull/15861
--
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:
users@infra.apache.org
[GitHub] [airflow] kaxil closed pull request #15861: Default `resultBackendConnection` to `metadataConnection`
Posted by GitBox <gi...@apache.org>.
kaxil closed pull request #15861:
URL: https://github.com/apache/airflow/pull/15861
--
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:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #15861: Default `resultBackendConnection` to `metadataConnection`
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #15861:
URL: https://github.com/apache/airflow/pull/15861#issuecomment-841478721
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest master or amend the last commit of the PR, and push it with --force-with-lease.
--
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:
users@infra.apache.org