ziqiangliang opened a new pull request, #7392:
URL: https://github.com/apache/gravitino/pull/7392
### What changes were proposed in this pull request?
- Replaced unsafe `doWithoutCommit[AndFetchResult]` methods with
`getWithoutCommit` in relational services
- Implemented proper connection/session resource cleanup using try-finally
blocks
- Modified affected methods in:
- `TableColumnMetaService.deleteColumnsByLegacyTimeline()`
- `TagMetaService#getTagForMetadataObject|associateTagsWithMetadataObject`
### Why are the changes needed?
Fix: #7378
Prevents database connection leaks that could lead to:
- Connection pool exhaustion
- Potential system instability under heavy load
- Resource management inconsistencies
### Does this PR introduce any user-facing change?
No
- Maintains identical transactional behavior
- Preserves all existing functionality
- Only changes internal resource management
### How was this patch tested?
- Connection pool stability verification under repeated operations (1000+
iterations)
- Transaction rollback behavior validation
- Query/update result correctness checks
- Integration with existing transaction management flows
--
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]
