Skip to content

Deprecate postgresql_lang#559

Merged
Andersson007 merged 5 commits intoansible-collections:mainfrom
elpavel:lang
Aug 11, 2023
Merged

Deprecate postgresql_lang#559
Andersson007 merged 5 commits intoansible-collections:mainfrom
elpavel:lang

Conversation

@elpavel
Copy link
Contributor

@elpavel elpavel commented Aug 9, 2023

SUMMARY

Deprecating the postgresql_lang module.

The lang module works only on old Postgresql versions.

The documentation says
As of PostgreSQL 9.1, most procedural languages have been made into “extensions”, and should therefore be installed with CREATE EXTENSION not CREATE LANGUAGE.

ISSUE TYPE
  • Module Deprecation Pull Request
COMPONENT NAME

postgresql_lang

ADDITIONAL INFORMATION

Fixes #519

Copy link
Collaborator

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@elpavel thanks for starting the process!

I think we should follow the official deprecation guide, could you please take a look?

Apologies for not mentioning this before

Copy link
Collaborator

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

one more small formatting things

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>
Comment on lines +289 to +290
module.warn("The postgresql_lang module has been deprecated and will be removed in community.postgresql 4.0.0")

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
module.warn("The postgresql_lang module has been deprecated and will be removed in community.postgresql 4.0.0")

i think we can remove it from here

@Andersson007
Copy link
Collaborator

@elpavel hm, it's strange, i see deprecations made same way in c.general, for example.. i'll ask the community

@Andersson007
Copy link
Collaborator

@elpavel just remove the hyphens, i didn't notice them.. thanks @felixfontein !

Copy link
Collaborator

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Andersson007 Andersson007 merged commit 5655558 into ansible-collections:main Aug 11, 2023
@patchback
Copy link

patchback bot commented Aug 11, 2023

Backport to stable-2: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2/56555587809dc4baef7dd0bf5f66cd3ff5915171/pr-559

Backported as #562

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@patchback
Copy link

patchback bot commented Aug 11, 2023

Backport to stable-1: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-1/56555587809dc4baef7dd0bf5f66cd3ff5915171/pr-559

Backported as #563

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Aug 11, 2023
* lang deprecated

* lang deprecated

* lang deprecated

* postgresql_lang

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>

* lang deprecated

---------

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>
(cherry picked from commit 5655558)
patchback bot pushed a commit that referenced this pull request Aug 11, 2023
* lang deprecated

* lang deprecated

* lang deprecated

* postgresql_lang

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>

* lang deprecated

---------

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>
(cherry picked from commit 5655558)
Andersson007 pushed a commit that referenced this pull request Aug 11, 2023
* lang deprecated

* lang deprecated

* lang deprecated

* postgresql_lang

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>

* lang deprecated

---------

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>
(cherry picked from commit 5655558)

Co-authored-by: Pavel Rabel <128324708+elpavel@users.noreply.github.com>
Andersson007 pushed a commit that referenced this pull request Aug 11, 2023
* lang deprecated

* lang deprecated

* lang deprecated

* postgresql_lang

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>

* lang deprecated

---------

Co-authored-by: Andrew Klychkov <aaklychkov@mail.ru>
(cherry picked from commit 5655558)

Co-authored-by: Pavel Rabel <128324708+elpavel@users.noreply.github.com>
@elpavel elpavel deleted the lang branch August 11, 2023 11:14
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.

Enable integration testing of postgresql_lang

4 participants