LuciferYang opened a new pull request, #13873: URL: https://github.com/apache/gravitino/pull/13873
### What changes were proposed in this pull request? - `added_equality_deletes` and `removed_equality_deletes` now take `addedEqualityDeletes()` / `removedEqualityDeletes()` instead of the equality delete file counters. - `getTimerResult` returns `totalDuration().toMillis()` instead of `count()`, so `total_duration_ms` and `total_planning_duration` hold milliseconds as the schema scripts describe. ### Why are the changes needed? The two record-count columns duplicated the file-count columns, and both duration columns stored the number of timed events, which is always 1. Rows written before this change keep their old values. Fix: #13869 ### Does this PR introduce _any_ user-facing change? Yes. New rows in `commit_metrics_report` and `scan_metrics_report` carry the actual equality delete record counts and durations in milliseconds. The schema is unchanged. ### How was this patch tested? `TestJdbcMetricsStore` used to compare each column with the same counter the store read, so it passed with the wrong mapping. It now checks literal values from the fixture: 18/20 equality delete records (the files are 5/8), 22 total, and 100000 / 600000 ms. -- 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]
