Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.0.M3
    • Fix Version/s: 2.0.0.M4
    • Component/s: Core
    • Labels:
      None

      Description

      Try to stop a job with SimpleJobOperator through JMX.

      A OptimisticLockingFailureException is thrown because the jobExecution version doesn't match with persisted one.

      Bug origin :
      When the JobExecution status is synchronized, the version is not updated.

        Issue Links

          Activity

          Hide
          Robert Kasanicky added a comment -

          Added version check before synchronizing JobExecution.status. If versions do not match both status and version are synchronized.

          Show
          Robert Kasanicky added a comment - Added version check before synchronizing JobExecution.status. If versions do not match both status and version are synchronized.
          Hide
          Dave Syer added a comment -

          What's the reason for the mismatch? Isn't it dangerous to just ignore it and synchronize?

          Show
          Dave Syer added a comment - What's the reason for the mismatch? Isn't it dangerous to just ignore it and synchronize?
          Hide
          Robert Kasanicky added a comment -

          JobOperator works with its own copy of JobExecution therefore the version mismatch between job's and jobOperators' version is inevitable. This issue fixed the obvious problem (when jobOperator updates the status, job's version becomes obsolete), but I think there can still be trouble with race conditions. I want to take a closer look at this in BATCH-961

          Show
          Robert Kasanicky added a comment - JobOperator works with its own copy of JobExecution therefore the version mismatch between job's and jobOperators' version is inevitable. This issue fixed the obvious problem (when jobOperator updates the status, job's version becomes obsolete), but I think there can still be trouble with race conditions. I want to take a closer look at this in BATCH-961

            People

            • Assignee:
              Dave Syer
              Reporter:
              Edouard Lemaistre
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 0.5d
                0.5d
                Remaining:
                Time Spent - 3h Remaining Estimate - 1h
                1h
                Logged:
                Time Spent - 3h Remaining Estimate - 1h
                3h