danny0405 opened a new pull request, #19351: URL: https://github.com/apache/hudi/pull/19351
### Describe the issue this Pull Request addresses The Flink client RowData/Avro conversion and row-format writer paths had substantial unit-test coverage gaps across primitive, nullable, nested, array, map, decimal, timestamp, serialization, and schema-conversion branches. These client-layer changes are intentionally scoped away from the Flink datasource sink/source work tracked separately under ENG-44419. ### Summary and Changelog Adds plain JUnit coverage in `hudi-client/hudi-flink-client` for: - RowData/Avro conversion round trips across supported primitive and logical types - Nullable, nested row, array, and map values - `HoodieFlinkInternalRowSerializer` serialize/deserialize round trips - Parquet RowData writer and schema conversion branches - RowData getters/setters, vector conversion, record accessors, and meta fields No production code was changed or copied. Local JaCoCo source-line coverage, measured from the same `upstream/master` baseline and final module test command: | Class | Before | After | | --- | ---: | ---: | | `AvroToRowDataConverters.java` | 23.4% | 90.3% | | `HoodieFlinkInternalRowSerializer.java` | 0.0% | 98.4% | | `HoodieFlinkLanceArrowUtils.java` | 85.3% | 85.3% | | `HoodieFlinkRecord.java` | 38.1% | 88.7% | | `HoodieSchemaConverter.java` | 82.6% | 86.0% | | `ParquetRowDataWriter.java` | 17.5% | 85.0% | | `ParquetSchemaConverter.java` | 72.2% | 87.3% | | `RowDataToAvroConverters.java` | 23.6% | 88.6% | | `RowDataUtils.java` | 21.2% | 93.8% | | `VectorConversionUtils.java` | 25.8% | 90.7% | Validation: ```text mvn -pl hudi-client/hudi-flink-client -Punit-tests -DskipITs -Drat.skip=true -Dcheckstyle.skip=true -Djacoco.skip=false test Tests run: 159, Failures: 0, Errors: 0, Skipped: 0 BUILD SUCCESS ``` ### Impact Test-only change. There is no public API, user-facing behavior, configuration, storage-format, or performance impact. ### Risk Level low The change adds deterministic client-layer unit tests and does not modify production code. The complete `hudi-flink-client` unit-test suite passes locally. ### Documentation Update none ### Contributor's checklist - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute) - [ ] Enough context is provided in the sections above - [ ] 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]
