代码精简

[mek1986]

1. 代码逻辑简化
2. .无用代码消除

• 合并了merge记录，重新提交

[lizongbo]

再核实一下你改动的原因，如果仅仅是简化待而引入风险，宁可不改。

[mek1986]

再核实一下你改动的原因，如果仅仅是简化待而引入风险，宁可不改。
这里有修复空指针：当holder==null，holder.getConnectionId()会导致运行异常
LOG.error("{conn-" + (holder != null ? holder.getConnectionId() : "null") + "} discard", error);

[lizongbo]

再核实一下你改动的原因，如果仅仅是简化待而引入风险，宁可不改。
这里有修复空指针：当holder==null，holder.getConnectionId()会导致运行异常
LOG.error("{conn-" + (holder != null ? holder.getConnectionId() : "null") + "} discard", error);

discard不是必然为true的。为什么要去掉？

[mek1986]

再核实一下你改动的原因，如果仅仅是简化待而引入风险，宁可不改。
这里有修复空指针：当holder==null，holder.getConnectionId()会导致运行异常
LOG.error("{conn-" + (holder != null ? holder.getConnectionId() : "null") + "} discard", error);

discard不是必然为true的。为什么要去掉？