-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
chore: bump postgres from 10 to 14 #19790
Conversation
Two more things I think we should do:
|
Codecov Report
@@ Coverage Diff @@
## master #19790 +/- ##
==========================================
+ Coverage 66.40% 66.54% +0.13%
==========================================
Files 1690 1692 +2
Lines 64732 64775 +43
Branches 6656 6661 +5
==========================================
+ Hits 42987 43103 +116
+ Misses 20046 19972 -74
- Partials 1699 1700 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a few non-blocking comments. And big thanks for adding the detailed docs - this piece of documentation has been sorely needed for a long time!
@@ -69,6 +69,32 @@ you can add the endpoints to `WTF_CSRF_EXEMPT_LIST`: | |||
WTF_CSRF_EXEMPT_LIST = [‘’] | |||
``` | |||
|
|||
### Using a production metadata database |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've started to prefer the term "metastore", as "metadata database" is a bit of a tongue twister. But we can do a full update of that in a follow-up PR.
Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
* chore: bump postgres from 10 to 14 * update helm chart * adding docs * fix docs * Update docs/docs/installation/configuring-superset.mdx Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com> * Update docs/docs/installation/configuring-superset.mdx Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com> * improve docs * improve docs Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
* chore: bump postgres from 10 to 14 * update helm chart * adding docs * fix docs * Update docs/docs/installation/configuring-superset.mdx Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com> * Update docs/docs/installation/configuring-superset.mdx Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com> * improve docs * improve docs Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
SUMMARY
Postgres 11.0 change log: https://www.postgresql.org/docs/release/11.0/
Postgres 12.0 change log: https://www.postgresql.org/docs/release/12.0/
Postgres 13.0 change log: https://www.postgresql.org/docs/release/12.0/
Postgres 14.0 change log: https://www.postgresql.org/docs/release/14.0/
Also updates helm postgres bitnami chart to 11.1.22, that sets postgres to 14.2:
https://github.com/bitnami/charts/blob/master/bitnami/postgresql/values.yaml#L90
ADDITIONAL INFORMATION