[#1678][0.9] fix(server): disk size leak on removing resources by AppPurgeEvent by zuston · Pull Request #1689 · apache/uniffle · GitHub
Skip to content

[#1678][0.9] fix(server): disk size leak on removing resources by AppPurgeEvent#1689

Merged
zuston merged 1 commit into
apache:branch-0.9from
zuston:branch-0.9
May 11, 2024
Merged

[#1678][0.9] fix(server): disk size leak on removing resources by AppPurgeEvent#1689
zuston merged 1 commit into
apache:branch-0.9from
zuston:branch-0.9

Conversation

@zuston

@zuston zuston commented May 11, 2024

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Descrease the disk size that calculated by local storage self on removing resources with AppPurgeEvent

Why are the changes needed?

Fix: #1678

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unit tests

What changes were proposed in this pull request?

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

Why are the changes needed?

(Please clarify why the changes are needed. For instance,

  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.)

Fix: # (issue)

Does this PR introduce any user-facing change?

(Please list the user-facing changes introduced by your change, including

  1. Change in user-facing APIs.
  2. Addition or removal of property keys.)

No.

How was this patch tested?

(Please test your changes, and provide instructions on how to test it:

  1. If you add a feature or fix a bug, add a test to cover your changes.
  2. If you fix a flaky test, repeat it for many times to prove it works.)

…pPurgeEvent (apache#1679)

### What changes were proposed in this pull request?

Descrease the disk size  that calculated by local storage self on removing resources with AppPurgeEvent

### Why are the changes needed?

Fix: apache#1678

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Unit tests
@zuston

zuston commented May 11, 2024

Copy link
Copy Markdown
Member Author

@zuston zuston merged commit cf84420 into apache:branch-0.9 May 11, 2024
@roryqi roryqi changed the title [BRANCH-0.9][#1678] fix(server): disk size leak on removing resources by AppPurge… [#1678][0.9] fix(server): disk size leak on removing resources by AppPurge… May 11, 2024
@roryqi roryqi changed the title [#1678][0.9] fix(server): disk size leak on removing resources by AppPurge… [#1678][0.9] fix(server): disk size leak on removing resources by AppPurgeEvent May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant