jiangxt2 commented on PR #11886: URL: https://github.com/apache/gravitino/pull/11886#issuecomment-5042441914
> * Can we fully recognize different column default values Yes — the getDefaultKinds() query to system.columns.default_kind covers all three kinds (DEFAULT, MATERIALIZED, ALIAS). For MATERIALIZED/ALIAS columns, we call toGravitino(typeBean, columnDef, true, nullable) with isExpression=true, which forces creation of an UnparsedExpression instead of attempting literal parsing. For DEFAULT columns, the existing code path through getBasicJdbcColumnInfo() is unchanged. There is a fourth kind EPHEMERAL (ClickHouse 21.12+) that is not explicitly handled, but such columns are excluded from SELECT * and are extremely rare in practice. If needed, we can add it in a follow-up. -- 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]
