-
Notifications
You must be signed in to change notification settings - Fork 20
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
[DPE-6249] Add max_locks_per_transaction
config option
#804
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #804 +/- ##
==========================================
- Coverage 75.55% 75.54% -0.01%
==========================================
Files 12 12
Lines 3105 3112 +7
Branches 464 465 +1
==========================================
+ Hits 2346 2351 +5
- Misses 620 621 +1
- Partials 139 140 +1 ☔ View full report in Codecov by Sentry. |
…r-transaction Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
max_locks_per_transaction
config option
👋🏻 We just merged a PR updating all typing syntax to Python 3.10+. Please remember to update this PR accordingly. |
…r-transaction Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Thanks for pointing that out, @sinclert-canonical! I updated this PR with the changes from the main branch. |
Issue
The
max_locks_per_transaction
PostgreSQL setting is not available for users to set.Solution
Create a config option to expose it to the users.