Davis-Zhang-Onehouse opened a new pull request, #19042:
URL: https://github.com/apache/hudi/pull/19042

   ### Change Logs
   
   A `HoodieTable` created per table-service work unit on the driver lazily 
builds a `FileSystemViewManager`. For a `SPILLABLE_DISK` view the file-group 
store spills to an on-disk `BitCaskDiskMap`. `HoodieTable` had no `close()`, 
and the per-cycle tables created in `BaseHoodieTableServiceClient` (`preCommit` 
/ `logCompact` / `compact` / `cluster` / `scheduleTableServiceInternal`) were 
never closed — so their cached views' on-disk maps lingered until JVM exit 
instead of being released promptly.
   
   - `HoodieTable` now implements `AutoCloseable`; `close()` releases the 
lazily-built `FileSystemViewManager` and the table metadata reader (idempotent, 
never throws).
   - The create-use-drop tables in `BaseHoodieTableServiceClient` are wrapped 
in try-with-resources.
   - Adds `TestHoodieTableResourceLifecycle` verifying `HoodieTable.close()` 
cleans up the spillable view's on-disk `BitCaskDiskMap` directory.
   
   ### Impact
   
   Promptly frees per-cycle driver-side file-system view resources (on-disk 
spillable maps) instead of leaving them until JVM exit. No behavior change to 
table-service results.
   
   ### Risk level: low
   
   Mechanical lifecycle change covered by existing file-system-view close tests 
plus the new test.
   
   ### Documentation Update
   
   None.
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Change Logs and Impact were stated clearly
   - [x] Adequate tests were added if applicable


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to