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.

Sign up for GitHub

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

Jump to bottom

Rework PG function guard #375

Merged

Y-- merged 1 commit into main from yl/pg-function-guard

Oct 30, 2024

Collaborator

Y-- commented Oct 30, 2024

Forked off #254

Rework the way we wrap calls to PG functions:

merge both flavor of PostgresFunctionGuard (returning void or not) into a single macro
drop the need for explicit template instantiation
automatically capture function name for better tracing

This PR doesn't systematically review all calls to PG functions, this will be addressed in a following one.


          Rework PG function guard

9c5a906

Y-- requested review from JelteF and mkaruza

October 30, 2024 14:54

JelteF approved these changes

View reviewed changes

Y-- merged commit 79941b4 into main

4 checks passed

Y-- deleted the yl/pg-function-guard branch

October 30, 2024 15:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet