[ 
https://issues.apache.org/jira/browse/FLINK-33022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17765421#comment-17765421
 ] 

Weihua Hu commented on FLINK-33022:
-----------------------------------

Resolved in master: b51ee30c3d1a212947398d880a676f07f46f36be

> Log an error when enrichers defined as part of the configuration can not be 
> found/loaded
> ----------------------------------------------------------------------------------------
>
>                 Key: FLINK-33022
>                 URL: https://issues.apache.org/jira/browse/FLINK-33022
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Runtime / Coordination
>    Affects Versions: 1.18.0
>            Reporter: Matt Wang
>            Assignee: Panagiotis Garefalakis
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.19.0
>
>
> if we configurate the `jobmanager.failure-enrichers`, but could not load the 
> class in 
> FailureEnricherUtils, no exceptions can be seen in the log currently, and it 
> is very inconvenient to check the problem. Here I suggest that some 
> ERROR-level logs should be added, or an exception should be thrown directly 
> (because the load cannot be uploaded is not an expected result)
> {code:java}
> // code placeholder
> @VisibleForTesting
> static Collection<FailureEnricher> getFailureEnrichers(
>         final Configuration configuration, final PluginManager pluginManager) 
> {
>     Set<String> includedEnrichers = 
> getIncludedFailureEnrichers(configuration);
>     LOG.info("includedEnrichers: {}", includedEnrichers);
>     //  When empty, NO enrichers will be started.
>     if (includedEnrichers.isEmpty()) {
>         return Collections.emptySet();
>     }
>     // TODO: here maybe load nothing
>     final Iterator<FailureEnricherFactory> factoryIterator =
>             pluginManager.load(FailureEnricherFactory.class);
>     ....
> } {code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to