incubator-uniffle icon indicating copy to clipboard operation
incubator-uniffle copied to clipboard

Published 20 hours ago • verified publisher icon apache

[#2188] fix(client): Fixup timeoutMs for executeTasks in ShuffleWriteClientImpl#sendAppHeartbeat

Open xumanbu opened this issue 4 months ago • 3 comments

What changes were proposed in this pull request?

(Please outline the changes and how this PR fixes the issue.)

Why are the changes needed?

The timeoutMs in ShuffleWriteClientImpl#sendAppHeartbeat method. The timeoutMs for per RPC is similar to the timeoutMs of ThreadUtils#executeTasks` for all servers RPC execute, and the logic here is flawed.

Resetting the timeoutMs for ThreadUtils.executeTask.

Fix: #2188

Does this PR introduce any user-facing change?

No.

How was this patch tested?

no need

xumanbu avatar Oct 16 '24 03:10 xumanbu