Added Configuration variables list to documentation #112
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
steve-chavez
approved these changes
Oct 18, 2023
Comment on lines
+158
to
+165
| You can change the variables by adding any of the following line to your postgres.conf file | ||
| ``` | ||
| pg_net.batch_size = <new integer> | ||
| pg_net.ttl = '<new ttl interval>' | ||
| pg_net.database_name = '<database name>' | ||
| ``` | ||
|
|
||
| After saving the file, you can execute `SELECT pg_reload_conf()` to update _postgres.conf_ for your database. If the extension does not respond to the update, it may be necessary to restart your database. |
There was a problem hiding this comment.
@TheOtherBrian1 By any chance did you tried doing this on Supabase? Or do you self host?
I think on Supabase right now you can't do ALTER SYSTEM .. plus pg_reload_conf() since they require superuser.
There was a problem hiding this comment.
I tested this on a self hosted instance. I just ran the following SQL to see how Supabase would react:
ALTER SYSTEM SET pg_net.batch_size = 200;It gave an error:
Error running SQL: Failed to run sql query: permission denied to set parameter "pg_net.batch_size"
Through Supabase's CLI, it is possible to override some config variables, but the permitted list does not extend to PG_NET.
I'll add a caveat to the documentation that configuration is only possible on self-hosted instances.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
Docs Update
What is the current behavior?
The documentation does not reference all the config variables.
Please link any relevant issues here:
Solves the primary complaint of of ISSUE #97
What is the new behavior?
The documentation outlines the configuration variables and how to update them.