Gabriel39 opened a new pull request, #67704:
URL: https://github.com/apache/doris/pull/67704

   ### What problem does this PR solve?
   
   Iceberg `FOR TIME AS OF` rejected timestamps containing fractional seconds, 
even though the snapshots metadata exposes `committed_at` with fractional 
precision. Truncating the value to whole seconds can also select the wrong 
snapshot when multiple snapshots are committed within the same second.
   
   ### What is changed and how does it work?
   
   - Accept an optional 1-9 digit fractional-second component in the 
Iceberg-specific time-travel parser while leaving other `TimeUtils` consumers 
unchanged.
   - Keep session time-zone interpretation unchanged.
   - Add unit coverage that distinguishes two snapshots committed within the 
same second.
   - Update the Iceberg time-travel regression matrix to feed the fractional 
`committed_at` value back into `FOR TIME AS OF` directly.
   
   ### Check List
   
   - [x] Unit test: `IcebergUtilsTest` (38 tests)
   - [x] FE Checkstyle
   - [x] `git diff --check`
   
   


-- 
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]

Reply via email to