venkata91 commented on PR #17562: URL: https://github.com/apache/iceberg/pull/17562#issuecomment-5803317291
@nastra @singhpk234 @manuzhang mind taking a look when you get a chance? This has been open a while and is ready for a committer review. It adds a `scanDuration` read metric alongside the existing `totalPlanningDuration`, following the same pattern as #15104. Approved by @anoopj, and review feedback from @uros-b has been addressed (the aggregate now renders as `total (min, med, max)` to match `MetricUtils.stringValue`, so straggler tasks stay visible). CI is green. #17820 builds on this with the write-side equivalent (`writeDuration` and `writeCloseDuration`). -- 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]
