JingsongLi commented on PR #9866: URL: https://github.com/apache/paimon/pull/9866#issuecomment-5708357150
Solid correctness fix. Verified the gate matches the mapper's lossiness exactly: the only lossy value mappers are the TIMESTAMP ones going through `Timestamp.toMicros()`, and `precision > 6` is precisely where that truncates. Precision 6 keeps answering value predicates (the second test pins that), and inheriting `REMAIN` for everything except `isNull`/`isNotNull` is the right conservative choice — `<>` and `NOT IN` were the dangerous ones because they flip the matched rows over the whole row count. The DE integration test comparing the indexed read with a full scan across the predicate shapes is a good safety net, including the guard that sub-microsecond values survive the write/read round trip. -- 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]
