gregoriusxu / booksleeve

Automatically exported from code.google.com/p/booksleeve
Other
0 stars 0 forks source link

Feature Request: IHashCommands.Set #31

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
It would be useful if the IHashCommands.Set method was overloaded to accept an 
IDictionary<string, string> property. A lot of the methods on RedisConnection 
have overloads for string and byte[] but this one is missing.

Also, the existing Set() method would be more useful if you passed in 
IDictionary rather than the concrete Dictionary type. We use 
ConcurrentDictionary a lot in multi-thread apps, which Booksleeve currently 
doesn't play nice with.

Thus, these two methods would be supported on IHashCommands:

IHashCommands.Set(int db, string key, IDictionary<string, string> values, bool 
queueJump = false)

IHashCommands.Set(int db, string key, IDictionary<string, byte[]> values, bool 
queueJump = false)

Dan..

Original issue reported on code.google.com by daniel.b...@xero.com on 15 Jan 2013 at 1:02

GoogleCodeExporter commented 8 years ago
This is addressed in StackExchange.Redis, the successor to BookSleeve

Original comment by marc.gravell on 20 Mar 2014 at 12:08