使用ServiceStack.Redis在高容量方案上发生意外回复

时间:2016-04-05 20:17:31

标签: c# multithreading redis servicestack.redis servicestack-bsd

我的问题与此问题非常相似:Protocol errors, "no more data" errors, "Zero length response" errors while using servicestack.redis in a high volume scenario

我在使用IIS的C#Web应用程序中使用ServiceStack v3.9.54.0。我可以在Redis版本2.8.17和3.0.501中看到错误。

我收到的错误如下:

ServiceStack.Redis.RedisResponseException: Unexpected reply: +PONG, sPort: 65197, LastCommand: GET EX:KEY:230
at ServiceStack.Redis.RedisNativeClient.CreateResponseError(String error)
at ServiceStack.Redis.RedisNativeClient.ParseSingleLine(String r)
at ServiceStack.Redis.RedisNativeClient.ReadData()
at ServiceStack.Redis.RedisNativeClient.SendExpectData(Byte[][] cmdWithBinaryArgs)
at ServiceStack.Redis.RedisNativeClient.GetBytes(String key)
at ServiceStack.Redis.RedisNativeClient.Get(String key)

ServiceStack.Redis.RedisResponseException: Unknown reply on integer response: 43PONG, sPort: 59017, LastCommand: EXISTS EX:AnKey:Cmp6
at ServiceStack.Redis.RedisNativeClient.CreateResponseError(String error)
at ServiceStack.Redis.RedisNativeClient.ReadLong()
at ServiceStack.Redis.RedisNativeClient.SendExpectLong(Byte[][] cmdWithBinaryArgs)
at ServiceStack.Redis.RedisNativeClient.Exists(String key)
at Redis.Documentos.RedisBaseType.Exists(String key)

我想到的第一件事就是我在多个线程上共享Redis连接,但我看不到PooledRedisClientManager的单例实现上的问题(Configs是一个静态类存储连接信息):

public class RedisProvider
{
    public PooledRedisClientManager Pool { get; set; }
    private RedisProvider()
    {

        var srv = new List<string> { $"{Configs.Server}:{Configs.Port}" };

        Pool = new PooledRedisClientManager(srv, srv, null, 
            Configs.Database, Configs.PoolSize, Configs.PoolTimeout);
    }

    public IRedisClient GetClient()
    {
        try
        {
            var connection = (RedisClient)Pool.GetClient();
            return connection;
        }
        catch (TimeoutException)
        {
            return null;
        }
    }

    private static RedisProvider _instance;
    public static object _providerLock = new object();
    public static RedisProvider Provider
    {
        get
        {
            lock (_providerLock)
            {
                if (_instance == null)
                {
                    var instance = new RedisProvider();
                    _instance = instance;
                    return _instance;
                }
                else
                {

                    return _instance;
                }
            }
        }
    }

}

所有客户都是通过池获得的,如下所示:

var redis = (RedisClient)RedisProvider.Provider.GetClient();

我确信redis var不是在多个线程之间共享的,据我所知,这段代码显示了正确的线程安全实现......

非常感谢任何帮助。

编辑:根据我使用的某些技术,我无法访问App Startup代码,也无法使用using块。所以,我把这样的客户包裹起来:

RedisClient redis;
try {
    redis = (RedisClient)RedisProvider.Provider.GetClient();
    // Do stuff
} finally {
    redis.Dispose();
}

1 个答案:

答案 0 :(得分:2)

此错误消息表示在多个线程之间共享相同的redis客户端实例,提供的源代码不提供任何不是的验证。

以上RedisProvider只是围绕单身人士的更详细的访问版本,例如:

public static class RedisProvider
{
    public static IRedisClientManager Pool { get; set; }

    public static RedisClient GetClient()
    {
        return (RedisClient)Pool.GetClient();
    }
}

RedisManager只需要在App Startup上初始化一次:

var srv = new List<string> { $"{Configs.Server}:{Configs.Port}" };
RedisProvider.Pool = new PooledRedisClientManager(srv, srv, null, 
    Configs.Database, Configs.PoolSize, Configs.PoolTimeout);

从那时起,详细锁定只会增加开销,并且与直接访问Singleton RedisManager相比,不提供任何线程安全优势。

虽然解析客户端是ThreadSafe:

var redis = RedisProvider.GetClient();

返回的redis客户端实例不是线程安全的(根据.NET约定)。因此,您需要确保不在多个线程之间共享同一实例,还需要确保客户端在使用后处理。

要确保在同一个线程中访问和处理它,您应该将客户端用法包装在using语句中:

using (var redis = RedisProvider.GetClient())
{
    //...
} 

如果您在需要使用RedisClient并且不在不同的后台线程,异步任务,并行化代码等中共享同一客户端实例时执行此操作,则不应再出现任何多线程问题。当您在不同的线程中需要新的客户端实例时,您应该使用相同的访问模式,并从池中检索(并配置)单独的客户端实例。