[GitHub] [camel] henka-rl commented on pull request #6267: CAMEL-17075 Moved initialisation from evaluate() to init()

[GitBox <gi...@apache.org>]

henka-rl commented on pull request #6267:
URL: https://github.com/apache/camel/pull/6267#issuecomment-942311351


   I run all core tests and SimpleBuilderTest fails, but I would say that's because they don't call init() before calling evaluate().
   If I look at the interfaces Predicate and Expression they both says that all initialization should be done in init() and should be thread safe:
   "A predicate should be thread-safe and be able to evaluate concurrently by different threads with different exchanges. Any initialization logic should be done by the init(CamelContext) method which allows to prepare the predicate such as wiring in resources, pre-parsing and what else."
   


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org