Skip to content

PCSM-312 connection pool size configurable - #63

Open
rasika-chivate wants to merge 35 commits into
mainfrom
PCSM-312-connection-pool-size-configurable
Open

rasika-chivate wants to merge 35 commits into
mainfrom
PCSM-312-connection-pool-size-configurable

Conversation

@rasika-chivate

Copy link
Copy Markdown
Collaborator

Add documentation for the connection pool size configuration.

For details, see the following ticket:

https://perconadev.atlassian.net/browse/PCSM-312

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation for configuring MongoDB client connection pool sizing via the maxPoolSize URI option, to help operators tune PCSM clone throughput and avoid pool-related bottlenecks.

Changes:

  • Documents maxPoolSize support and where to set it (--source/--target URIs or PCSM_SOURCE_URI/PCSM_TARGET_URI).
  • Provides URI syntax examples and describes default/unlimited behaviors.
  • Adds operational recommendations relating pool size to clone worker configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/install/parameters.md Outdated
Comment thread docs/install/parameters.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.

Comment thread docs/install/parameters.md Outdated
Comment thread docs/install/parameters.md Outdated
Comment thread docs/install/parameters.md Outdated
Comment thread docs/install/parameters.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@nastena1606
nastena1606 temporarily deployed to PCSM-312-connection-pool-size-configurable - plm-docs PR #63 June 26, 2026 08:10 — with Render Destroyed
@rasika-chivate
rasika-chivate temporarily deployed to PCSM-312-connection-pool-size-configurable - pcsm-docs PR #63 June 26, 2026 08:10 — with Render Destroyed
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@nastena1606
nastena1606 temporarily deployed to PCSM-312-connection-pool-size-configurable - plm-docs PR #63 June 26, 2026 09:16 — with Render Destroyed
@rasika-chivate
rasika-chivate temporarily deployed to PCSM-312-connection-pool-size-configurable - pcsm-docs PR #63 June 26, 2026 09:16 — with Render Destroyed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.

Comment thread docs/install/parameters.md
Comment thread docs/install/parameters.md
Comment thread docs/install/parameters.md
Comment thread docs/install/parameters.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@nastena1606
nastena1606 temporarily deployed to PCSM-312-connection-pool-size-configurable - plm-docs PR #63 June 26, 2026 10:48 — with Render Destroyed
rasika-chivate and others added 2 commits June 26, 2026 16:18
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread docs/install/parameters.md
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread docs/install/parameters.md

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Comment thread docs/install/parameters.md Outdated
Co-authored-by: Adnan <chupe@chupe.ba>
Comment on lines +83 to +92
2026-06-24 15:16:40.691 INF Percona ClusterSync for MongoDB v0.10.0 3eb82dd 2026-06-24_09:36_UTC
2026-06-24 15:16:40.692 INF Config: source client compressors: [snappy zstd zlib] s=connect
2026-06-24 15:16:40.692 INF Config: source client maxPoolSize: 500 s=connect
2026-06-24 15:16:40.711 INF Connected to source cluster [Percona Server for MongoDB 8.0.16-5]: mongodb://rs00:30000
2026-06-24 15:16:40.711 INF Config: target client compressors: [snappy zstd zlib] s=connect
2026-06-24 15:16:40.711 INF Config: target client maxPoolSize: 100 (driver default) s=connect
2026-06-24 15:16:40.724 INF Connected to target cluster [Percona Server for MongoDB 8.0.16-5]: mongodb://rs10:30100
2026-06-24 15:16:40.728 INF Checking Recovery Data for "pcsm" s=recovery
2026-06-24 15:16:40.729 INF Recovery Data not found s=recovery
2026-06-24 15:16:40.729 INF Starting HTTP server at http://localhost:2242

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These samples use the pre-0.10.0 timestamp format. #69 switches PCSM to RFC 3339 UTC and restamps every log sample in the docs, and this section is marked 0.10.0, so these would ship stale. Can we restamp these and any other examples in the right format eg. 2026-06-24T15:16:40.691Z?

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.

5 participants