Skip to content

fix(c/driver/postgresql): make connection init/release thread-safe - #4938

Merged
lidavidm merged 3 commits into
apache:mainfrom
jtanx:cwd
Oct 30, 2026
Merged

lidavidm merged 3 commits into
apache:mainfrom
jtanx:cwd

Conversation

@jtanx

@jtanx jtanx commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Closes #4753

@jtanx
jtanx requested a review from lidavidm as a code owner October 21, 2026 08:50

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

IMO, not necessary

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Happy to remove, but is it not necessary to mention? The default docs imply to assume serialized access unless documented otherwise.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's overly wordy, this doesn't make it into user-facing docs, and it's placed as if it's the most important thing about the driver.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure, have deleted

Comment thread c/driver/postgresql/database.cc Resolved
@lidavidm
lidavidm merged commit aa2c1c2 into apache:main Oct 30, 2026
96 of 101 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.

c/driver/postgresql: make connection init/release thread safe

2 participants