jamesnetherton commented on code in PR #24065:
URL: https://github.com/apache/camel/pull/24065#discussion_r3426459646
##########
dsl/camel-yaml-dsl/camel-yaml-dsl-common/src/main/java/org/apache/camel/dsl/yaml/common/YamlDeserializationContext.java:
##########
@@ -120,14 +141,204 @@ protected Optional<ConstructNode>
findConstructorFor(Node node) {
@Override
public void start() {
+ loadResolvers();
+ }
+
+ private void loadResolvers() {
ObjectHelper.notNull(camelContext, "camel context");
-
this.resolvers.addAll(getCamelContext().getRegistry().findByType(YamlDeserializerResolver.class));
+ if (resolversLoaded) {
+ return;
+ }
+
+ addResolverEntries(loadResolversFromClasspath());
Review Comment:
Could we perhaps have some way for runtimes to suppress loading resolvers
from the classpath?
On Camel Quarkus it's not really needed because we can discover and
configure all `YamlDeserializerResolver` impls at build time.
--
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]
