[FLINK-30183] We should add a proper error message in case the deprecated reflection-based instantiation of a reporter is triggered

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.16.0
• Fix Version/s: 1.16.1
• Component/s: Runtime / Metrics
• Labels: pull-request-available starter
• Assignee: Matthias Pohl
• Reporter: Matthias Pohl
• Created: 24/Nov/22 05:01
• Updated: 29/Nov/22 06:13
• Resolved: 29/Nov/22 06:13

Description

A user reported an issue with the PrometheusReporter instantiation. The NoSuchMethodException is caused by the PromethusReporter not having a default constructor implemented. Instead, the user should switch to the factory class approach.

We might want to log a warning if the reflection-based approach still works but wrap the NoSuchMethodException into a IllegalConfigurationException with a proper error message pointing to the factory-based approach

Issue Links

• is caused by: FLINK-27206 Deprecate reflection-based reporter instantiation
• links to: GitHub Pull Request #21387