jiangxt2 opened a new pull request, #11915: URL: https://github.com/apache/gravitino/pull/11915
Fix: #11910 ## What changes were proposed in this pull request? - Fix GraphiteMergeTree incorrectly classified as not requiring ORDER BY (single-arg → three-arg constructor) - Add Enum8/Enum16 normalization via startsWith guard before the type switch - Add type mappings for Int128/Int256/UInt128/UInt256 (→ ExternalType, preserves round-trip) - Add BFloat16 → ExternalType mapping (no half-precision in Gravitino) - Restore DATE32 → ExternalType mapping (Date32 has wider range than Date) ## Why are the changes needed? GraphiteMergeTree tables cannot be created through Gravitino at all — the engine requires ORDER BY but the catalog rejects it. Multiple ClickHouse types (Enum8/16, wide integers, BFloat16, Date32) fall through to generic ExternalType instead of being handled explicitly. ## Does this PR introduce _any_ user-facing change? No. GraphiteMergeTree tables become creatable; other type mappings preserve existing ExternalType behavior. ## How was this patch tested? - `./gradlew :catalogs-contrib:catalog-jdbc-clickhouse:test -PskipITs` — all 46 unit tests pass - Type converter round-trip tests cover Enum8/Enum16, Int128/Int256/UInt128/UInt256, BFloat16, DATE32 -- 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]
