jamesnetherton commented on issue #9421:
URL: https://github.com/apache/camel-quarkus/issues/9421#issuecomment-6582112045

   > @jamesnetherton was dropping unnamed beans intended in 
https://github.com/apache/camel-quarkus/pull/7559, or a consequence of avoiding 
null map keys?
   
   At the time, it was probably because of my reasoning around bean 'naming'. I 
assumed with Quarkus you either give a bean an explicit `@Name` or 
`@Idnetifier`. If those are not present, then the bean should be ignored due to 
`null` map keys.
   
   But standalone Camel and Spring can apply a 'default' name to registry 
beans, so I guess we should perhaps mimic this with the `bean.getIdentifier()` 
fallback.
   
   Maybe the simplest thing to do is open a PR and see if anything breaks. If 
we decide to go ahead, we'd need a migration guide note to capture the 
behaviour change for users.


-- 
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]

Reply via email to