Skip to content
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

Add citus.local_shared_pool_size #989

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

onderkalaci
Copy link
Member

With Citus 10, we add a new GUC which implements connection throttling
for local node. This commit adds the GUC to the reference.

With Citus 10, we add a new GUC which implements connection throttling
for local node. This commit adds the GUC to the reference.
@jonels-msft jonels-msft temporarily deployed to citus-doc-pr-add-local--jqkifi March 2, 2021 13:09 Inactive

Specifies the maximum number of connections, across all simultaneous
sessions, is allowed to make to the local node. This setting is relevant
when the local node has shards, such as Citus on Single Node.
Copy link
Member Author

Choose a reason for hiding this comment

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

What is the syntax to give a reference toCitus on Single Node section?

Copy link
Member

Choose a reason for hiding this comment

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

:ref:`Citus on Single Node <development>`

or

:ref:`development`

if you want to use the text of the linked header, in this case Single-Node Citus.

Copy link
Member

@jonels-msft jonels-msft left a comment

Choose a reason for hiding this comment

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

Sorry @onderkalaci ! I added a review comment long ago, but never submitted the review!

I thought you weren't responding but it was my fault.

@@ -385,7 +385,7 @@ citus.max_shared_pool_size (integer)
************************************

Specifies the maximum number of connections that the coordinator node, across
all simultaneous sessions, is allowed to make per worker node. PostgreSQL must
all simultaneous sessions, is allowed to make per remote node. PostgreSQL must
Copy link
Member

Choose a reason for hiding this comment

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

Is this terminology something we want to generally change? Should probably keep it consistent...

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.

2 participants