Inject MetricsPersistence directly into PersistingMetricsReporter

[dimas-b]

This is a rebased version of #4708 from @bowsii (plus conflict resolutions)

Fixes #4655

1. Added a CDI producer for MetricsPersistence in ServiceProducers.

2. Removed CallContext dependency from PersistingMetricsReporter.

3. Removed PolarisMetaStoreManager dependency from
   PersistingMetricsReporter.

4. Injected MetricsPersistence directly into
   PersistingMetricsReporter.

5. Replaced:
   metaStoreManager.writeScanMetrics(...)
   with:
   metricsPersistence.writeScanReport(...)

6. Replaced:
   metaStoreManager.writeCommitMetrics(...)
   with:
   metricsPersistence.writeCommitReport(...)

7. Updated Javadocs to reflect direct MetricsPersistence usage.

8. Refactored PersistingMetricsReporterTest:

   • Removed CallContext mocks.
   • Removed PolarisMetaStoreManager mocks.
   • Added MetricsPersistence mock.
   • Updated verifications to use
     writeScanReport() and writeCommitReport().

Checklist

• 🛡️ Don't disclose security issues! (contact security@apache.org)
• 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
• 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
• 💡 Added comments for complex logic
• 🧾 Updated CHANGELOG.md (if needed)
• 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

[snazy]

LGTM!

I wonder whether PolarisEventManager would be the next one?