63ab38f917
### What changes were proposed in this pull request? This is a followup of https://github.com/apache/spark/pull/32534 , and proposes to free the memory entry immediately instead of doing it in the backround asynchronously. The reason is: 1. It's a bit weird to free the resource in an asynchronous way. 2. We free the off-heap memory entry in the same thread, and it's better to be consistent with it. 3. We can simplify the code quite a bit. This PR also simplifies the tests to reuse the class definition. ### Why are the changes needed? code simplification ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? existing tests Closes #32743 from cloud-fan/follow. Lead-authored-by: Wenchen Fan <cloud0fan@gmail.com> Co-authored-by: Wenchen Fan <wenchen@databricks.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com> |
||
|---|---|---|
| .. | ||
| benchmarks | ||
| src | ||
| pom.xml | ||