You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Once one app's all shuffleIds has been unregistered by the ShufflePurgeEvent, the above code will be invalid, because shuffleToCachedBlockIds will be empty and them won't trigger the removeResource by app level. So the app folder still remains after expire.
Affects Version(s)
master
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?
Yes I am willing to submit a PR!
The text was updated successfully, but these errors were encountered:
zuston
added a commit
to zuston/incubator-uniffle
that referenced
this issue
Apr 10, 2024
### What changes were proposed in this pull request?
Always executing the removeResources no matter whether blockIds remain.
### Why are the changes needed?
Fix: #1634
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Using existing unit tests, but additionally checking the app folder after app is expired.
### What changes were proposed in this pull request?
Always executing the removeResources no matter whether blockIds remain.
### Why are the changes needed?
Fix: #1634
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Using existing unit tests, but additionally checking the app folder after app is expired.
Code of Conduct
Search before asking
Describe the bug
I found some shuffle-servers basic dir has lots of app folders remains wich has been expired.
After digging this case, I think this is caused by the children shuffleIds unregistering. Please see here: https://github.com/apache/incubator-uniffle/blob/a7a0b4ce41a213f8f3e05a1e1198c41857706229/server/src/main/java/org/apache/uniffle/server/ShuffleTaskManager.java#L778-#L781 .
Once one app's all shuffleIds has been unregistered by the
ShufflePurgeEvent
, the above code will be invalid, becauseshuffleToCachedBlockIds
will be empty and them won't trigger the removeResource by app level. So the app folder still remains after expire.Affects Version(s)
master
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?
The text was updated successfully, but these errors were encountered: