thswlsqls commented on PR #8609: URL: https://github.com/apache/paimon/pull/8609#issuecomment-5044914730
Thanks for the review. You're right — the parser can only emit `TIME(3)` (Debezium sends `io.debezium.time.Time` as int32 millis for every `time(0..3)`), while the JDBC schema path keeps `TIME(n)`, and `canConvert()` had no TIME widening (only TIMESTAMP), so `TIME(0) -> TIME(3)` still threw. Added TIME precision widening to `canConvert` mirroring the TIMESTAMP branch — `time(0..2)` now converge on `TIME(3)` — with a test, and reworded the title. Just pushed. -- 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]
