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

doc: clarify connection pooling in createClient and fix broken link in isolationPoolOptions #2896

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

Conversation

rickyssj2
Copy link

Description

This PR improves the createClient documentation by:

  • Adding a Connection Pooling section to clarify that a single connection is usually sufficient for Redis and directing users to RedisClientPool for scenarios requiring connection pooling.
  • Updating isolationPoolOptions to provide a clearer explanation and replacing the broken link with a reference to createClientPool.
  • These updates help address #2845.

…in isolationPoolOptions

- Added a Connection Pooling section in `createClient` documentation to clarify that a single connection is typically sufficient and to provide guidance on when to use a connection pool.
- Updated `isolationPoolOptions` description with a more precise explanation and replaced the broken link with a reference to `createClientPool`.
- Changes made based on issue redis#2845.
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.

1 participant