一、系统异常现象
系统有一个功能向别的系统多线程推送用户数据信息，前几天发现该推送功能报内部错误，经过查看后台日志文件，发现org.redisson.client.RedisConnectionException: Unable to connect to Redis server:，io.netty.channel.ChannelException: Unable to create Channel from class class io.netty.channel.socket.nio.NioSocketChannel。
错误日志输出特别清晰明了，显示redis的连接已经满了，不能创建新的NioSocketChannel了。
二、问题分析
在该功能用户redis的地方无非就两个。
第一个场景是多线程请求其他系统的token值，这个过程中加了锁，确保多线程只有一个线程拿到token值并且将该token写入redis，其他线程复用这个线程获取的token进行用户数据推送，这样可以降低双方系统的资源消耗。
2、第二个场景就是在redis里面取出上一步的token值，进行下一步的业务处理。
关键代码：
public String generateToken(RedissonClient redissonClient) {
RLock lock = redissonClient.getLock(InterfacesConstants.REDISSON_LOCK);
try {
lock.lock(30, TimeUnit.SECONDS);
RBucket
//获取推送用户的token值
String pushUserToken = testBucket.get();
log.info("线程名为：{}的线程获取到「pushUserToken」为：{}", Thread.currentThread().getName(), pushUserToken);
if (EmptyUtil.isEmpty(pushUserToken)) {
//获取token
IDasTokenResultDto tokenFromIDas = getTokenFromIDas();
//获取IAM平台的token值
pushUserToken = tokenFromIDas.getAccess_token();
if (EmptyUtil.isEmpty(pushUserToken)) {
log.info("未接收到IAM返回的token值！！");
return null;
}
//重置token过期时间
testBucket.set(pushUserToken, (Integer.parseInt(tokenFromIDas.getExpires_in())) / 1000 - 5, TimeUnit.SECONDS);
}
//拼接IAM所需要的token值
String finalAccessToken = "Bearer " + pushUserToken;
return finalAccessToken;
} catch (Exception e) {
log.error("获取IAM平台的token值错误，error message：{}", e.getMessage());
} finally {
lock.unlock();
}
return null;
}
仔细检查上述场景的业务代码，也没有发现类似的问题。。。。。。
回过头又仔细分析了整个业务链条的场景，终于发现了问题所在。问题出在了RedissonClient实例化上，一般的项目会把这种公共的类交给spring容器去自动实例化，这个老项目因为架构的原因开发者自己去实例的该对象。
所以每次触发这个功能都会实例一次RedissonClient对象，从而导致redis的连接越来越来多并没有释放，一直到沾满。所以只要在RedissonClient实例的时候做点小动作就可以填上这个坑了。
原代码：
@Slf4j
public class RedissonConfig {
private static final String REDIS_CONFIG_FILE = "Config/redis.properties";
private static final String SINGLE = "standalone";
private static final String CLUSTER = "cluster";
private static final String HOST_PORT = "hostports";
private RedissonConfig() {
throw new IllegalStateException("Utility class");
}
public static RedissonClient initialRedissonClient() {
Map
if(EmptyUtil.isEmpty(redisInfoMap)){
return null;
}
String runMode = redisInfoMap.get("runMode");
RedissonClient redisson=null;
//单机
if(Objects.equals(SINGLE,runMode)){
redisson = initialSingleRedisson();
log.info("实例化单节点「RedissonClient」成功！！");
//集群
}else if (Objects.equals(CLUSTER,runMode)){
redisson = initialClusterRedisson();
log.info("实例集群「RedissonClient」成功！！");
}
return redisson;
}
public static RedissonClient initialSingleRedisson(){
try {
Map
if(EmptyUtil.isEmpty(redisInfoMap)){
return null;
}
String hostPort = redisInfoMap.get(HOST_PORT);
String redisPwd = redisInfoMap.get("pwd");
//单机
Config config = new Config();
config.useSingleServer().setAddress("redis://" + hostPort).setPassword(redisPwd);
RedissonClient redisson = Redisson.create(config);
log.info("「SingleRedisson」初始化成功！！");
return redisson;
}catch (Exception e){
log.info("「SingleRedisson」初始化异常！！message：{}",e.getMessage());
e.printStackTrace();
}
return null;
}
public static RedissonClient initialClusterRedisson(){
//集群
try {
Map
if(EmptyUtil.isEmpty(redisInfoMap)){
return null;
}
String hostPort = redisInfoMap.get(HOST_PORT);
String redisPwd = redisInfoMap.get("pwd");
Config config = new Config();
String[] hostPostArr = hostPort.split(",");
for (String str : hostPostArr) {
config.useClusterServers()
//设置扫描间隔时间
.setScanInterval(2000)
.setPassword(redisPwd)
.addNodeAddress("redis://" + str);
}
RedissonClient redisson = Redisson.create(config);
return redisson;
}catch (Exception e) {
log.info("「ClusterRedisson」初始化异常！！message：{}",e.getMessage());
e.printStackTrace();
}
return null;
}
public static Map
try {
//加载连接池配置文件
Properties props = PropsUtil.loadProps(REDIS_CONFIG_FILE);
String runMode = props.getProperty("redis.runMode");
String hostPort = props.getProperty("redis.hostports");
String redisPwd = props.getProperty("redis.password");
Map
map.put("runMode",runMode);
map.put(HOST_PORT,hostPort);
map.put("pwd",redisPwd);
log.info("解析redis配置文件获取到的连接信息：{}",map);
if(EmptyUtil.isEmpty(map)){
log.warn("未解析到redis的配置信息！！");
return Maps.newHashMap();
}
return map;
}catch (Exception e) {
log.info("redisson初始化获取结点和密码失败");
e.printStackTrace();
}
return Maps.newHashMap();
}
}
三、问题解决
知道问题在哪了，剩下的就好说了，我们只要保证该Java进程里面实例化有且只有一个RedissonClient对象就可以，spring的单例bean还用不了，这就得造一波轮子了。都到这了，单例模式走起。。。
优化后代码：
@Slf4j
public class RedissonConfig {
private static final String REDIS_CONFIG_FILE = "Config/redis.properties";
private static final String SINGLE = "standalone";
private static final String CLUSTER = "cluster";
private static final String HOST_PORT = "hostports";
/**
* volatile修饰（防止指令重排序）
**/
private volatile static RedissonClient instance;
private RedissonConfig() {
throw new IllegalStateException("Utility class");
}
public static RedissonClient initialRedissonClient() {
//单例模式的双重校验
if(instance == null){
log.info("「RedissonClient」实例instance为空，进行实例化操作");
synchronized (RedissonConfig.class) {
//如果是空，就实例化对象
if (instance == null) {
Map
if(EmptyUtil.isEmpty(redisInfoMap)){
return null;
}
String runMode = redisInfoMap.get("runMode");
//单机
if(Objects.equals(SINGLE,runMode)){
instance = initialSingleRedisson();
log.info("实例化单节点「RedissonClient」成功！！");
//集群
}else if (Objects.equals(CLUSTER,runMode)){
instance = initialClusterRedisson();
log.info("实例集群「RedissonClient」成功！！");
}
}
}
}else {
log.info("「RedissonClient」实例instance为：{},无需重新实例化「RedissonClient」",instance);
}
return instance;
}
public static RedissonClient initialSingleRedisson(){
try {
Map
if(EmptyUtil.isEmpty(redisInfoMap)){
return null;
}
String hostPort = redisInfoMap.get(HOST_PORT);
String redisPwd = redisInfoMap.get("pwd");
//单机
Config config = new Config();
config.useSingleServer().setAddress("redis://" + hostPort).setPassword(redisPwd);
RedissonClient redisson = Redisson.create(config);
log.info("「SingleRedisson」初始化成功！！");
return redisson;
}catch (Exception e){
log.info("「SingleRedisson」初始化异常！！message：{}",e.getMessage());
e.printStackTrace();
}
return null;
}
public static RedissonClient initialClusterRedisson(){
//集群
try {
Map
if(EmptyUtil.isEmpty(redisInfoMap)){
return null;
}
String hostPort = redisInfoMap.get(HOST_PORT);
String redisPwd = redisInfoMap.get("pwd");
Config config = new Config();
String[] hostPostArr = hostPort.split(",");
for (String str : hostPostArr) {
config.useClusterServers()
//设置扫描间隔时间
.setScanInterval(2000)
.setPassword(redisPwd)
.addNodeAddress("redis://" + str);
}
RedissonClient redisson = Redisson.create(config);
return redisson;
}catch (Exception e) {
log.info("「ClusterRedisson」初始化异常！！message：{}",e.getMessage());
e.printStackTrace();
}
return null;
}
public static Map
try {
//加载连接池配置文件
Properties props = PropsUtil.loadProps(REDIS_CONFIG_FILE);
String runMode = props.getProperty("redis.runMode");
String hostPort = props.getProperty("redis.hostports");
String redisPwd = props.getProperty("redis.password");
Map
map.put("runMode",runMode);
map.put(HOST_PORT,hostPort);
map.put("pwd",redisPwd);
log.info("解析redis配置文件获取到的连接信息：{}",map);
if(EmptyUtil.isEmpty(map)){
log.warn("未解析到redis的配置信息！！");
return Maps.newHashMap();
}
return map;
}catch (Exception e) {
log.info("redisson初始化获取结点和密码失败");
e.printStackTrace();
}
return Maps.newHashMap();
}
}
对比以前的代码只需要实例化的时候改成单例模式就行了。
四、总结
出现类似问题的原因可能一般开发者潜移默化的依赖spring架构的便利性，以至于一些特殊场景的实现不知觉中就按照以前的编码风格去开发，前期简单测试不会发现问题所在，系统压力上来了以后才能暴露出来。程序猿：我不管，反正是测试的锅，谁让他们压测做的不到位呢！！测试：我尼玛。。。。。
精彩内容
您阅读本篇文章共花了：
发表评论