sundapeng commented on PR #8999: URL: https://github.com/apache/paimon/pull/8999#issuecomment-5165897350
One lossy path remains: TINYINT/SMALLINT still accept any physical INT32. The reader casts out-of-range values (for example, 128 -> -128 and 65535 -> -1), so a predicate on the converted value can still prune the row group using the original INT32 value. Please allow INT32 pushdown only when it has a signed IntLogicalTypeAnnotation with bit width <= 8 for TINYINT or <= 16 for SMALLINT. For bare INT32 or a wider/incompatible annotation, throw UnsupportedOperationException to drop the predicate. Please also add regression tests for the two examples above. -- 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]
