[
https://issues.apache.org/jira/browse/CAMEL-7627?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14209893#comment-14209893
]
Jonathan Anstey edited comment on CAMEL-7627 at 11/13/14 3:23 PM:
------------------------------------------------------------------
Just merged your patches with a few minor modifications. Thanks Nikolay!
was (Author: janstey):
Just merged your patches with a few miro modifications. Thanks Nikolay!
> Quartz/Quartz2 in cluster mode doesn't apply changed trigger settings
> ---------------------------------------------------------------------
>
> Key: CAMEL-7627
> URL: https://issues.apache.org/jira/browse/CAMEL-7627
> Project: Camel
> Issue Type: Bug
> Components: camel-quartz, camel-quartz2
> Reporter: Nikolay Turpitko
> Assignee: Jonathan Anstey
> Fix For: 2.14.1, 2.15.0, 2.13.4
>
> Attachments: 0001-Fix-reshedule-changed-trigger-after-restart.patch,
> 0002-Test-trigger-type-change.patch
>
>
> Camel-quartz2 component in clustered mode uses trigger options stored in DB
> rather (possibly changed) ones from endpoint's URI.
> Desirable behavior is to compare trigger options in DB and endpoint's URI and
> reschedule quartz job when they changed (like in camel-quartz component).
> Component camel-quartz already have this functionality, but there is no test
> for it and it works incorrectly with changed SimpleTrigger options.
> I attached a patch with unit tests. Every test prepares DB, than creates
> application context twice with different trigger options. Both times it
> retrieves options back, accessing them via trigger (not via endpoint, so that
> it uses values stored in DB). After that it asserts that retrieved options
> are indeed different.
> You can ensure, that the tests fail with old versions of
> org.apache.camel.component.quartz2.QuartzEndpoint#addJobInScheduler or
> org.apache.camel.component.quartz.QuartzComponent#hasTriggerChanged methods
> and pass with patched implementation.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
