Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Postgres profile configuration to reference retries parameter #1733

Closed
1 task done
tomasfarias opened this issue Jul 18, 2022 · 1 comment
Closed
1 task done
Labels
content Improvements or additions to content dbt Core The changes proposed in this issue relate to dbt Core improvement Use this when an area of the docs needs improvement as it's currently unclear

Comments

@tomasfarias
Copy link

Contributions

  • I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates

Postgres profile: https://docs.getdbt.com/reference/warehouse-profiles/postgres-profile
And Redshift profile (as the underlying adapter is the same): https://docs.getdbt.com/reference/warehouse-profiles/redshift-profile

What part(s) of the page would you like to see updated?

dbt-core PR #5432 resolved dbt-core issue #5022 by adding retry functionality for dbt-postgres and dbt-redshift (as it inherits it from the former).

The docs should be changed so that users are aware of the existence of a new retries configuration in the profiles.yml of Postgres and Redshift, and that they can tweak it according to their needs.

The change should add retries to the sample profiles.yml for Postgres and Redshift, mention the default of 1, and explain how this parameter determines how many times will dbt attempt to connect a Postgres/Redshift database, which could help with transient network issues, like latency spikes.

Additional information

The PR is still in development, but very close to finished, so the doc changes should line up with the dbt-core version that includes the change.

@tomasfarias tomasfarias added content Improvements or additions to content improvement Use this when an area of the docs needs improvement as it's currently unclear labels Jul 18, 2022
@runleonarun runleonarun added the dbt Core The changes proposed in this issue relate to dbt Core label May 15, 2023
@runleonarun
Copy link
Collaborator

I see retries documented in both postgres and redshift profiles so I am closing this issue, but feel free to open if I've closed it in error!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content dbt Core The changes proposed in this issue relate to dbt Core improvement Use this when an area of the docs needs improvement as it's currently unclear
Projects
None yet
Development

No branches or pull requests

2 participants