yihua opened a new pull request, #19163:
URL: https://github.com/apache/hudi/pull/19163

   ### Change Logs
   
   Adds targeted test coverage for the Spark SQL write/commit path in 
hudi-spark-common, exercised end-to-end through Spark SQL from the hudi-spark 
test module:
   
   - `TestMergeIntoWriteCoverage` — MERGE INTO conditional update / 
delete-marker / conditional insert on partitioned COW and MOR tables 
(row-writer enabled and disabled), plus an overwrite-payload record-merge case. 
Covers `ExpressionPayload` (matched/not-matched evaluators, delete markers, MOR 
getInsertValue/isDeleted, doRecordMerge branches) and `MergeIntoKeyGenerator` 
(record-key / partition-path extraction and key-generator fallback).
   - `TestBulkInsertRowWriterCommitCoverage` — row-writer `bulk_insert` and 
INSERT OVERWRITE (dynamic partitions, static partition, and rejection when 
overlapping a pending clustering plan). Covers 
`BaseDatasetBulkInsertCommitActionExecutor` and 
`DatasetBulkInsertOverwriteCommitActionExecutor`.
   - `TestAlterTableColumnCoverage` — schema-on-read column 
add/type-change/rename/comment/drop on an unpartitioned table, and rejection of 
primary-key/ordering-field changes. Covers `AlterTableCommand`.
   
   All tests assert concrete outcomes (written rows, post-alter schema, merge 
results including delete-marker cases); no assertion-free 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]

Reply via email to