thswlsqls opened a new pull request, #9558: URL: https://github.com/apache/paimon/pull/9558
### Purpose fix #9557 - `toLiteral()` made DATE an `i32` literal and precision-0 TIMESTAMP a MILLISECONDS one, while the writer records `vortex.date[days]` / `vortex.timestamp[s]`. Vortex does not implicitly cast, so the scan fails with `Cannot compare scalars with incompatible types`. `millis / 1000` matches `ArrowUtils.nonCastedTimestampToEpoch()`. - Origin: `77670e7ea` had a `precision <= 0` branch marked `// seconds`. `609745ec1` added `TimeUnit.SECONDS`/`DAYS`/`literalDate`, moved DECIMAL to `literalDecimal`, but folded that branch into `precision <= 3`, leaving DATE and precision 0 behind. - These filters now push down, joining the existing `VortexRecordsReader.returnedPosition` mismatch that already affects other types under deletion vectors / row lineage (separate issue). ### Tests - Added three round-trip cases to `VortexPredicateConverterTest` (DATE `>=`, TIMESTAMP(0) `>=`, TIMESTAMP_LTZ(0) `=`) reusing its `roundTrip(...)` helper. All fail on master. - `mvn -pl paimon-vortex/paimon-vortex-format -DfailIfNoTests=false clean install` on JDK 11 / darwin-aarch64 — 67 tests, 0 failures/errors, 4 skipped (`JavaPyVortexE2ETest`, CI-covered); checkstyle/spotless/enforcer passed. -- 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]
