nevzheng opened a new pull request, #11984: URL: https://github.com/apache/gravitino/pull/11984
### What changes were proposed in this pull request? Adds `testCreateAndLoadIcebergTableV2Types` to `CatalogIcebergBaseIT`. It creates a table containing the V2 column types that were not previously exercised end-to-end — `boolean`, `float`, `double`, `uuid`, `fixed`, `binary`, `decimal`, `list`, `map` — through the native metadata API, loads it back, and asserts that (a) the Gravitino types round-trip unchanged and (b) the underlying Iceberg schema stores the expected Iceberg types. Because these are all valid V2 types, the test runs on both backends via the existing subclasses (`CatalogIcebergRestIT`, `CatalogIcebergHiveIT`) with no assumptions. ### Why are the changes needed? `CatalogIcebergBaseIT` only round-tripped a subset of V2 types through a live backend (`int`, `long`, `date`, `string`, `struct`, `time`, `timestamp`). The converter mappings for the remaining types are unit-tested in `TestConvertUtil`, but were never verified through the real REST/IRC and Hive backends — so backend/serialization regressions for `decimal` (precision/scale), `fixed` (length), `list`, and `map` in particular could slip through. This closes that gap. Fix: #11983 ### Does this PR introduce _any_ user-facing change? No. Test-only. ### How was this patch tested? This is the test. It compiles cleanly (`:catalogs:catalog-lakehouse-iceberg:compileTestJava` → `BUILD SUCCESSFUL`). As a Docker IT it runs under `CatalogIcebergRestIT` / `CatalogIcebergHiveIT` (`-PskipDockerTests=false`). -- 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]
