xiangfu0 commented on PR #19118: URL: https://github.com/apache/pinot/pull/19118#issuecomment-5171161411
Addressed in `e07d2974e9`. The reserved field is now `$upsertComparisonTime`, so Pinot’s existing `$` hidden-column contract excludes it from `SELECT *` in both query engines while allowing explicit reads. I kept the preprocessing approach: legacy segments materialize the field as a physical LONG from the source segment creation time, and rewritten segments preserve physical per-row values. The updated Refresh E2E verifies physical materialization, transactional replacement ordering, wildcard hiding, and explicit timestamp reads. It passed through the full 63-module reactor (956 Maven goals); the focused 46-test reactor and all required formatting/style/license gates also pass. Ref: https://github.com/apache/pinot/pull/19118#issuecomment-5170523641 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
