q8webmaster commented on PR #8238: URL: https://github.com/apache/paimon/pull/8238#issuecomment-4703650995
Closing — the fix is in the wrong layer for master. On master, `LongTimestampUpdater.longTimestamp()` already normalises MICROS-annotated INT64 values to epoch-milliseconds before storing them in the `LongColumnVector`, so `ParquetTimestampVector.getTimestamp()` always receives epoch-ms for precision≤3 and `fromEpochMillis` is correct. The E2E failures confirmed this. The reader bug exists in paimon 1.4.1 where `LongTimestampUpdater` lacks `timestampUnit()` and stores raw file values — but that version is not the right target for this fix. -- 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]
