Open
Conversation
Contributor
Author
|
如果可以话能改成ILogger最好。。。。。 |
Owner
|
这几个输出要考虑兼容.net4.0，回头我测试下。 另外，他们不是疯狂打印，一般问题不大，可能不优雅。 日志可以参考 FreeRedis，是重新开发的，预留了事件。 |
Contributor
Author
|
我之前是用的StackExchange.Redis，由于并发问题改为CSRedis，我们自身对CSRedis做了一些改造用于支持哨兵切换的情况😑 |
Contributor
Author
|
之前哨兵的问题我们做了一些宕机测试，其实CSRedis和FreeRedis都不能很好支持。。。 |
Owner
|
FreeRedis 每个模式都支持的，基于 redis6.0 开发，支持更广。 |
Contributor
Author
|
各种模式倒是没问题，主要连接哨兵以后，在运行过程中出现主从切换的话，这两个都不能很好支持，特别是在Docker环境下，出现网络异常或Redis节点宕机的时候可能会导致更严重的问题 |
Owner
|
期待，感谢贡献 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
由于Console输出太多东西，期望改成Trace输出方便扩展