suryaprasanna opened a new pull request, #19051:
URL: https://github.com/apache/hudi/pull/19051
### Describe the issue this Pull Request addresses
Adds missing test coverage for clean scheduling: when multiple cleans are
disabled (`hoodie.clean.multiple.enabled=false`) and a clean is already
pending, `clean()` should execute the pending clean instead of scheduling a
new one. Also removes a redundant variable in an existing test.
### Summary and Changelog
- Add `testExecutingPendingCleanInstantsBeforeSchedulingNewCleanInstant` to
`TestHoodieClientOnCopyOnWriteStorage`.
- Remove the redundant `engineContext` from
`TestHoodieSerializableFileStatus`
(use the inherited `context`; drop now-unused imports).
### Impact
None. Test-only change; no production code or public API is affected.
### Risk Level
none — only test code.
### Documentation Update
none
### Contributor's checklist
- [x] Read through [contributor's
guide](https://hudi.apache.org/contribute/how-to-contribute)
- [x] Enough context is provided in the sections above
- [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]
