yihua opened a new pull request, #19164: URL: https://github.com/apache/hudi/pull/19164
### Change Logs Adds focused unit tests for several small, previously untested (or near-zero-coverage) classes in hudi-spark-datasource, chipping away at the long tail of low-coverage files: - `CachingIterator` (hudi-spark-common): hasNext idempotency and full iteration. - `HoodieSparkCatalogUtils.MatchBucketTransform`: bucket-transform extraction (match / non-match). - `BasicStagedTable`: delegation to the underlying Table, abort/commit staged changes, and rejection of non-writable LogicalWriteInfo (was 0%). - `HoodieSparkValidateDuplicateKeyRecordMerger`: merge throws HoodieDuplicateKeyException with the offending key; merging strategy id and pre-combining fallback (was 0%). - `ProcedureParameter`/`ProcedureParameterImpl`: required/optional factories, accessors, hashCode, and toString. - `org.apache.hudi.cli.SchemaProvider`: default target=source behavior, HoodieSchema wrapping, null handling, and the getTargetSchema fallback path (was 0%). All tests assert real behavior; no bare-instantiation tests and no sleeps. ### Impact Test-only. No production code changes. ### Risk level none ### Documentation Update None. ### Contributor's checklist - [x] Read through the contributor's guide - [x] Change is test-only and behavior-preserving - [x] Tests assert meaningful behavior -- 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]
