8000 made idle timeout settable by mhanney · Pull Request #183 · ServiceStack/ServiceStack.Redis · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Dec 24, 2022. It is now read-only.

made idle timeout settable#183

Merged
mythz merged 1 commit intoServiceStack:masterfrom
mhanney:master
Sep 19, 2013
Merged

made idle timeout settable#183
mythz merged 1 commit intoServiceStack:masterfrom
mhanney:master

Conversation

@mhanney
Copy link
@mhanney mhanney commented Sep 19, 2013

I am using ServiceStack.Redis PooledRedisClientManager with redistogo.com. Redistogo closes idle client connections after 150 seconds (on the small plan). The current default idle timeout in ServiceStack.Redis is hard set at 240 seconds. This results in "No more data" exceptions when using a connection from the pool that has been closed by the server. Lowering the idle time out to < 150 (for redistogo) fixes this because RedisNativeClient.AssertConnectedSocket does the necessary checks and reconnects the client.

@mythz
Copy link
Member
mythz commented Sep 19, 2013

Can you approve the Contributor License Agreement so we can include this, thx.

@mhanney
Copy link
Author
mhanney commented Sep 19, 2013

This is done,

Thanks,
Mike.

On Thu, Sep 19, 2013 at 3:09 PM, Demis Bellot notifications@github.comwrote:

Can you approve the Contributor License Agreementhttps://docs.google.com/forms/d/16Op0fmKaqYtxGL4sg7w_g-cXXyCoWjzppgkuqzOeKyk/viewformso we can include this, thx.


Reply to this email directly or view it on GitHubhttps://github.com//pull/183#issuecomment-24777126
.

......... __ o
.........-<,
......(O) (O)...........
...........................

mythz added a commit that referenced this pull request Sep 19, 2013
made idle timeout settable
@mythz mythz merged commit be37f20 into ServiceStack:master Sep 19, 2013
@mythz
Copy link
Member
mythz commented Sep 19, 2013

great, thx.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0