search

michaelklishin / quartz-mongodb

A MongoDB-based store for the Quartz scheduler. This fork strives to be as feature complete as possible. Originally by MuleSoft.
Other
249 stars 203 forks source link

Trigger jobDataMap is being ignored #21

Closed pizerg closed 11 years ago

pizerg commented 11 years ago

This implementation does not take into account jobData defined in triggers, only the one defined in Jobs.

michaelklishin commented 11 years ago

In what method? RAMJobStore#storeJob, for example, does not merge trigger data map into the job's one. It does not even have the trigger in the context.

It may be Quartz's responsibility to merge the two data maps together before passing it on to the store.

michaelklishin commented 11 years ago

Inspecting the reference implementation suggests it does not perform job data map merging. So it must happen in Quartz core, possibly after both job and trigger a loaded from the store.