dimas-b commented on code in PR #3468: URL: https://github.com/apache/polaris/pull/3468#discussion_r2714035430
########## CHANGELOG.md: ########## @@ -52,6 +52,8 @@ request adding CHANGELOG notes for breaking (!) changes and possibly other secti - The EclipseLink Persistence implementation has been completely removed. - The default request ID header name has changed from `Polaris-Request-Id` to `X-Request-ID`. - The (Before/After)CommitTableEvent has been removed. +- The `PolarisMetricsReporter.reportMetric()` method signature has been extended to include a Review Comment: As far as the CHANGELOG "diff" in this PR is concerned, it looks correct to me. I'd expect @pingtimeout (as the release manager for 1.3.0) to open a reconciliation PR for `main` and add a dedicated section for `1.3.0` soon :wink: I'm sure @pingtimeout will be able to deal with conflicts if this PR merges first :slightly_smiling_face: -- 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]
