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

[Improvement] Release preAllocatedBuffer when unregisterShuffleByAppId request

Open maobaolong opened this issue 1 year ago • 0 comments

Code of Conduct

Search before asking

  • [X] I have searched in the issues and found no similar issues.

What would you like to be improved?

Release memory when unregisterShuffleByAppId invoked, rather than release it by checkResource background thread.

[2024-07-23 15:16:14.233] succeeded=true        allowed=true    cmd=unregisterShuffleByAppId    statusCode=SUCCESS      appId=application_1703049085550_16568008_1721718824735  shuffleId=-1    executionTimeUs=96     args:{null}
[2024-07-23 15:18:28.153] [checkResource-0] [WARN] ShuffleTaskManager - Remove expired preAllocatedBuffer[id=24467] that required by app: application_1703049085550_16568008_1721718824735
[2024-07-23 15:18:28.153] [checkResource-0] [WARN] ShuffleTaskManager - Remove expired preAllocatedBuffer[id=24468] that required by app: application_1703049085550_16568008_1721718824735

How should we improve?

No response

Are you willing to submit PR?

  • [x] Yes I am willing to submit a PR!

maobaolong avatar Jul 23 '24 07:07 maobaolong