nicolaferraro opened a new pull request #223: Fix #222: initial implementation 
of native cron support
URL: https://github.com/apache/camel-k-runtime/pull/223
 
 
   Fix #222
   
   This is a basic impl that needs some help to be improved.
   
   I didn't go the source-loader way because source loaders cannot be easily 
mixed and this means technically e.g. that you cannot use the cron feature in a 
knative-source, because both require a distinct source loader (delegation is 
basic currently).
   
   I've added two customizers, one for `timer` and one for `quartz`. Dependency 
to the respective camel libraries is optional, so loading a customizer does not 
take transitive dependencies of the other.
   
   The customizer replaces options at endpoint level. This means that the 
Endpoint definition (that sometimes is printed in the logs) contains the 
original URI, but the endpoint values will be different at runtime. I don't 
know how much this is ok..
   
   There's also a route policy that shuts the route down once the first 
exchange is completed (gracefully, to give time to all dependent exchanges to 
complete). This may not be sufficient in the Quarkus case as you mentioned 
(@davsclaus , @lburgazzoli ), because shutting down the context does not take 
down the application, right?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to