Song Jiacheng has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21041 )

Change subject: [Java] Fix concurrent problem while traversing the traces of 
KuduRpc.
......................................................................


Patch Set 9:

(2 comments)

> Patch Set 8:
>
> (2 comments)
>
> Thank you for the fix!

Thanks for the review!

http://gerrit.cloudera.org:8080/#/c/21041/8//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/21041/8//COMMIT_MSG@10
PS8, Line 10: HashedWheelTimer.
> HashedWheelTimer
Done


http://gerrit.cloudera.org:8080/#/c/21041/8//COMMIT_MSG@12
PS8, Line 12: which leads to
            : exception in timeout task
> Could you add an example of such a trace into the commit description?  It c
Sure, i have added the stacktrace into the commit message.
I think it should be the late response. I took a quick look at the method 
addTrace() and it could only be called by response at that time.



--
To view, visit http://gerrit.cloudera.org:8080/21041
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1a642d93d01ebcfa05b01fe263023b5580d2542b
Gerrit-Change-Number: 21041
Gerrit-PatchSet: 9
Gerrit-Owner: Song Jiacheng <songjiach...@thinkingdata.cn>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Song Jiacheng <songjiach...@thinkingdata.cn>
Gerrit-Comment-Date: Thu, 22 Feb 2024 06:29:31 +0000
Gerrit-HasComments: Yes

Reply via email to