sohutv / cachecloud

搜狐视频(sohu tv)Redis私有云平台 :支持Redis多种架构(Standalone、Sentinel、Cluster)高效管理、有效降低大规模redis运维成本,提升资源管控能力和利用率。平台提供快速搭建/迁移,运维管理,弹性伸缩,统计监控,客户端整合接入等功能。(CacheCloud is a Redis cloud management platform. It supports Standalone, Sentinel, and Cluster architectures for Redis, effectively reducing large-scale Redis operation and maintenance costs, and improving resource management and utilization. The platform provides rapid construction/migration, operation and maintenance management, elastic scaling, statistical monitoring, client integration and access and other functions)
http://cachecloud.github.io/
Apache License 2.0
8.91k stars 2.04k forks source link

这部分代码是不是有问题?一个变量完全没用到 #108

Closed Calabor-Hoc closed 7 years ago

Calabor-Hoc commented 7 years ago

https://github.com/sohutv/cachecloud/blob/7e9b5482385597f66e694c227713dbc80f1ff5a3/cachecloud-open-web/src/main/java/com/sohu/cache/redis/impl/RedisCenterImpl.java#L474

这里用for循环做的 accMap.putAll(rowMap)完全没有用处啊,accMap的值不会被函数返回,函数执行完就扔掉了。是不是作者想做什么没做成?

Calabor-Hoc commented 7 years ago

看错了,不好意思😅