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

Upgrading postgres to 15 in build #6989

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

davidwatkins73
Copy link
Member

No description provided.

@davidwatkins73
Copy link
Member Author

@TL41 - are you okay with us bumping up the postgres version to 15 (from 11) ? The gen_random_uuid() function is failing on the earlier version. We think this could be worked around by explicitly adding an extension - however 11 is getting a bit old and 15 works out the box.

@davidwatkins73 davidwatkins73 added this to the done : 1.58 milestone Feb 19, 2024
@davidwatkins73 davidwatkins73 added fixed (test & close) An issue has been fixed, merged into master and is ready for further testing and removed fixed (test & close) An issue has been fixed, merged into master and is ready for further testing labels Feb 19, 2024
@davidwatkins73 davidwatkins73 removed this from the done : 1.58 milestone Feb 19, 2024
@TL41
Copy link
Contributor

TL41 commented Feb 19, 2024

@TL41 - are you okay with us bumping up the postgres version to 15 (from 11) ? The gen_random_uuid() function is failing on the earlier version. We think this could be worked around by explicitly adding an extension - however 11 is getting a bit old and 15 works out the box.

@davidwatkins73 - I don't have any immediate concerns so it should be fine. Any steps we would need to follow as far as installs and updates?

@davidwatkins73
Copy link
Member Author

I don't think there are any specific steps. However if your postgres version is older than 15 (14 should also be okay) then you may not be able to run the ddl. If that is so let us know and we can revert this change and implement a workaround for the gen_random_uuid() function.

@davidwatkins73 davidwatkins73 merged commit f51d4e4 into master Feb 21, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants