rionmonster commented on code in PR #28437: URL: https://github.com/apache/flink/pull/28437#discussion_r3707946757
########## flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/TypeInferenceUtil.java: ########## Review Comment: Good question @Timm0! I confirmed (and had my Claude-shaped assistant verify) that there should be behavior change here. `ARRAY<DECIMAL(2,2)>` literals still silently truncate on cast, same as before. The check only should only apply to a top-level `DECIMAL` argument; arrays never reach it since their expected type is computed from their own elements, not the outer target. I went ahead and updated the related Javadocs to call out container types explicitly so this doesn't come up again. -- 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]
