dimas-b commented on code in PR #3385:
URL: https://github.com/apache/polaris/pull/3385#discussion_r2854945189


##########
CHANGELOG.md:
##########
@@ -58,6 +58,24 @@ request adding CHANGELOG notes for breaking (!) changes and 
possibly other secti
 - Relaxed `client_id`, `client_secret` regex/pattern validation on reset 
endpoint call
 - Added support for S3-compatible storage that does not have KMS (use 
`kmsUavailable: true` in catalog storage configuration)
 - Added support for storage-scoped AWS credentials, allowing different AWS 
credentials to be configured per named storage. Enable with the 
`RESOLVE_CREDENTIALS_BY_STORAGE_NAME` feature flag (default: false). Storage 
names can be set explicitly via the `storageName` field on storage 
configuration, or inferred from the first allowed location's host.
+- Added support for persisting Iceberg metrics to a separate database using 
named datasources. Metrics can now be stored in the same database (different 
schema) or a completely separate database from entity data. To enable metrics 
persistence:
+  1. Configure a named datasource `metrics` in your application properties:
+     ```properties
+     # Same database, different schema
+     
quarkus.datasource.metrics.jdbc.url=jdbc:postgresql://localhost:5432/polaris?currentSchema=metrics

Review Comment:
   nit: this may be too verbose for a CHANGELOG entry. Could we keep the 
example in `application.properties` and just reference it from the CHANGELOG?



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

Reply via email to