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 minn connection per node property #159

Conversation

mrozk
Copy link
Contributor

@mrozk mrozk commented Nov 4, 2022

No description provided.

@mrozk
Copy link
Contributor Author

mrozk commented Nov 4, 2022

Is it possible to create a patch for version 0.7.1, because we haven't upgrade aerospike version yet, it would be great to have this config for older version of aerospike client

@mrozk
Copy link
Contributor Author

mrozk commented Nov 4, 2022

/**
 * Minimum number of synchronous connections allowed per server node.  Preallocate min connections
 * on client node creation.  The client will periodically allocate new connections if count falls
 * below min connections.
 * <p>
 * Server proto-fd-idle-ms and client {@link ClientPolicy#maxSocketIdle} should be set to zero
 * (no reap) if minConnsPerNode is greater than zero.  Reaping connections can defeat the purpose
 * of keeping connections in reserve for a future burst of activity.
 * <p>
 * Default: 0
 */
public int minConnsPerNode;

@ijusti
Copy link

ijusti commented Nov 7, 2022

@roimenashe can you please merge it

@roimenashe roimenashe added the enhancement New feature or request label Nov 8, 2022
@roimenashe roimenashe merged commit b9db0e5 into aerospike-community:main Nov 9, 2022
roimenashe pushed a commit that referenced this pull request Nov 9, 2022
Co-authored-by: abandura <abandura@playtika.com>
(cherry picked from commit b9db0e5)
@roimenashe
Copy link
Collaborator

@mrozk @ijusti
Released 2 new versions:
0.8.1 (latest) https://github.com/aerospike-community/spring-data-aerospike-starters/releases/tag/0.8.1
0.7.2 https://github.com/aerospike-community/spring-data-aerospike-starters/releases/tag/0.7.2

Both contain this change.
It might take up to 24 hours to sync in Maven Central.

@mrozk
Copy link
Contributor Author

mrozk commented Nov 9, 2022

Thank you so much guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants