[Improvement] Methods like `updateFileset` need to be done with in a transaction or lock scope.
Open
improvement
- Dominant language
- Java
- Stars
- 3.2k
- Forks
- 935
- Avg merge
- 1d 16h
- Merged PRs (30d)
- 298
Description
### What would you like to be improved?
https://github.com/apache/gravitino/blob/0b767848b0cd36ceb3eaaa3749d73a551aa56cc1/core/src/main/java/org/apache/gravitino/storage/relational/service/FilesetMetaService.java#L195-L214
Please see the above logic; the whole updates(get, update) should be done within a transaction.
### How should we improve?
_No response_
Contributor guide
Research direction
Start with core/src/main/java/org/apache/gravitino/storage/relational/service/FilesetMetaService.java around lines 195-214 and trace the updateFileset get/update sequence. Done means the complete update is performed within a transaction or lock scope, with tests demonstrating the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- backend, databases
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100