[GitHub] [dubbo] zrlw edited a comment on issue #8923: 建议github构建workflow增加unit-test失败上传log4j日志的配置

[GitBox <gi...@apache.org>]

zrlw edited a comment on issue #8923:
URL: https://github.com/apache/dubbo/issues/8923#issuecomment-927315477


   testServerHeartbeat方法挂起不动弹这个现象可以手工重现：
   修改HeartbeatHandlerTest测试类，把testServerHeartbeat方法复制一下搞两个测试方法，代码完全相同(改不改端口号，结果都是如此），将其中一个方法名改掉，比如改成testServerHeartbeatClone，然后把这个测试类的其他测试方法注释掉，然后执行HeartbeatHandlerTest测试，这两个方法，谁在后面运行谁就会hang住，跟踪了一下发现是处理connected事件时，并没有去调用HeartbeatHandler的connected方法，channel对象的ReadTimestamp和WriteTimestamp属性一直都是null，CloseTimerTask每次执行doTask任务时lastRead和lastWrite方法都是返回null，导致无法判断空闲时间是否超时。
   感觉是HeartbeatHandler只能bind成功一次，这个代码还是2年前提交的，不知道原来就是这个样，还是多实例触发的问题。
   @AlbumenJ @chickenlj 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org