Skip to content

Commit

Permalink
MINOR: Optimize log printing format for AbstractHerder#validateConver…
Browse files Browse the repository at this point in the history
…terConfig (#18199)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
  • Loading branch information
YunKuiLu authored Dec 30, 2024
1 parent 4080f19 commit af7f403
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -469,7 +469,7 @@ private <T> ConfigInfos validateConverterConfig(
VersionRange range = PluginUtils.connectorVersionRequirement(pluginVersion);
pluginInstance = (T) plugins().newPlugin(pluginClass, range, connectorLoader);
} catch (VersionedPluginLoadingException e) {
log.error("Failed to load {} class {} with version {}: {}", pluginName, pluginClass, pluginVersion, e);
log.error("Failed to load {} class {} with version {}", pluginName, pluginClass, pluginVersion, e);
pluginConfigValue.addErrorMessage(e.getMessage());
pluginVersionValue.addErrorMessage(e.getMessage());
return null;
Expand All @@ -479,7 +479,7 @@ private <T> ConfigInfos validateConverterConfig(
return null;
} catch (InvalidVersionSpecificationException e) {
// this should have been caught by prior validation logic
log.error("Invalid version range for {} class {}: {}", pluginName, pluginClass, pluginVersion, e);
log.error("Invalid version range for {} class {} with version {}", pluginName, pluginClass, pluginVersion, e);
pluginVersionValue.addErrorMessage(e.getMessage());
return null;
}
Expand Down

0 comments on commit af7f403

Please sign in to comment.