将Console输出改为Trace输出

2881099 / csredis

.NET Core or .NET Framework 4.0+ client for Redis and Redis Sentinel (2.8) and Cluster. Includes both synchronous and asynchronous clients.

MIT License

2.01k stars 414 forks source link

Open EsonXie opened 1 year ago

EsonXie commented 1 year ago

由于Console输出太多东西，期望改成Trace输出方便扩展

EsonXie commented 1 year ago

如果可以话能改成ILogger最好。。。。。

2881099 commented 1 year ago

这几个输出要考虑兼容.net4.0，回头我测试下。

另外，他们不是疯狂打印，一般问题不大，可能不优雅。

日志可以参考 FreeRedis，是重新开发的，预留了事件。

EsonXie commented 1 year ago

我之前是用的StackExchange.Redis，由于并发问题改为CSRedis，我们自身对CSRedis做了一些改造用于支持哨兵切换的情况😑 如果FreeRedis能够支持哨兵切换，并且能够让我们平滑过渡的话，我肯定会换的😁

EsonXie commented 1 year ago

之前哨兵的问题我们做了一些宕机测试，其实CSRedis和FreeRedis都不能很好支持。。。所以我在外层用CSRedis订阅了哨兵节点，然后每分钟都输出了Hello的消息

2881099 commented 1 year ago

FreeRedis 每个模式都支持的，基于 redis6.0 开发，支持更广。

EsonXie commented 1 year ago

各种模式倒是没问题，主要连接哨兵以后，在运行过程中出现主从切换的话，这两个都不能很好支持，特别是在Docker环境下，出现网络异常或Redis节点宕机的时候可能会导致更严重的问题关于哨兵集群主从切换，以及可用性这些问题，我之后有时间还是想贡献到FreeRedis上的😁

2881099 commented 1 year ago

期待，感谢贡献