suxiaogang223 opened a new pull request, #61381:
URL: https://github.com/apache/doris/pull/61381
### What problem does this PR solve?
Iceberg execute actions still have two validation and semantics gaps:
1. `rollback_to_timestamp` declares support for both formatted timestamps
and epoch millis, but the execution path only parses `yyyy-MM-dd HH:mm:ss.SSS`.
When epoch millis is passed, Doris converts it to `-1` and calls
`rollbackToTime(-1)`.
2. `rewrite_data_files` does not reject invalid file size bounds where
`min-file-size-bytes > max-file-size-bytes`, so illegal input is not failed
fast in FE with a clear message.
This PR fixes both issues.
### What changes are included in this PR?
- Support epoch millis parsing in `rollback_to_timestamp` execution, while
keeping the existing formatted timestamp path unchanged
- Fail fast in `rewrite_data_files` validation when `min-file-size-bytes >
max-file-size-bytes`
- Add regression coverage for:
- `rollback_to_timestamp` with epoch millis input
- `rewrite_data_files` invalid min/max file size bounds
### How was this patch tested?
- `bash run-regression-test.sh --run -d external_table_p0/iceberg/action -s
test_iceberg_optimize_actions_ddl`
In the current local environment, the framework runs successfully but the
Iceberg suite is skipped because `enableIcebergTest` is not enabled, so the new
Iceberg cases were not executed locally.
Issue Number: N/A
Related Jira:
- DORIS-24683
- DORIS-24712
--
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]
