[CAMEL-14883] Make CamelBeanPostProcessor implement Ordered

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.2.0
• Fix Version/s: 3.3.0
• Component/s: camel-spring
• Estimated Complexity: Unknown
• Assignee: Unassigned
• Reporter: Eickvonder
• Created: 12/Apr/20 00:08
• Updated: 14/Apr/20 06:06
• Resolved: 14/Apr/20 06:06

Description

The CamelBeanPostProcessor does not implement the Ordered interface provided by Spring.

According to Spring documentation it is recommended to do so:

https://docs.spring.io/spring/docs/current/spring-framework-reference/core.html#beans-factory-extension-bpp

If you write your own BeanPostProcessor, you should consider implementing the Ordered interface, too

And not doing so may break certain things (see e.g. https://github.com/spring-projects/spring-framework/issues/24058) but Spring guys won't fix that to keep backwards-compatibility. So I would suggest to let the CamelBeanPostProcessor implement the Ordered interface, e.g. with lowest priority or with configurable priority. The actual value is irrelevant, the issue described in https://github.com/spring-projects/spring-framework/issues/24058 only occurs if a post processor exists that does not implement the ordered interface.
 

Issue Links

• links to: GitHub Pull Request #3731