nicolaferraro commented on issue #223: Fix #222: initial implementation of 
native cron support
URL: https://github.com/apache/camel-k-runtime/pull/223#issuecomment-572514373
 
 
   > > Good point, but would you remove all quartz libraries on the operator 
side?
   > 
   > Yes, I think in case we want to leverage k8s scheduler we can replace the 
discovered dependencies with what make more sense for us so in this case quartz 
--> timer
   > 
   > > I did minimal variations to make sure an user can switch from 
kube-scheduled mode to standard mode easily (potentially even without changes 
in the library path).
   > 
   > Guess this can be controlled by a trait so the `auto` mode would do the 
magic but if you need fine grained control, then you can use the trait to 
influence the level of the magic
   > 
   > > I've overridden the required fields, keeping other user settings on the 
endpoints.. even if I can't find something relevant that an user may customize 
on quartz. There are some headers added on the exchanges, that can be faked in 
case.
   > 
   > I think that if you want k8s jobs, then you do not care about headers 
generated by the camel components, if you want them, probably you need to keep 
your pod running as the headers won't be what you expect (i.e. the number of 
fired event will allays be just 1)
   
   Yeah, I was referring to `quartz' that has a fire time... but that can be 
taken from system time.
   
   Another benefit of the approach is that you can make it generic as it can 
replace all (ENV var) given endpoints with timers. And that literally allows 
you to plug new endpoints, directly from the operator.. even inventing new 
ones.. like `kubernetes-cron:0/1 * * * *`.
   
   Let me do another attempt.. do you think `Runtime.Listener` is a good place 
to start?

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