Connection pooling and some cleanup/fixes

stchang / redis

A redis client for Racket.

17 stars 4 forks source link

Closed m4burns closed 9 years ago

m4burns commented 10 years ago

This pull request adds:

A connection pool implementing thread-safe leasing of automatically created connections.
Some minor fixes to make defcmd/.. work when commands are executed inside a MULTI/EXEC block.
Wrapping the body of with-redis-connection in dynamic-wind such that the connection is closed regardless of how control escapes the body (exception, captured continuation, ...).

I am also planning to write some tests and add them here.

stchang commented 9 years ago

Thanks again for this. Looking at it now.

stchang commented 9 years ago

Ok I agree this is better. Thanks for supporting the single connections as well though. Maybe I'll phase it out at some point.