Uploaded image for project: 'jBPM'
  1. jBPM
  2. JBPM-7837

ExecutorServiceImpl.requeue(Long olderThan) doesn't calculate a correct value

    Details

    • Sprint:
      2018 Week 39-41
    • Story Points:
      2
    • Workaround:
      Workaround Exists
    • Workaround Description:
      Hide
      • Set system property "org.kie.executor.timeunit" to "MILLISECONDS"
      • Do not forget to set related properties. e.g. "org.kie.executor.running.max" to "600000"
      • In case of jBPM 6, you would also need to set "org.kie.executor.interval" to "3000", "org.kie.executor.initial.delay" to "100000"
      Show
      Set system property "org.kie.executor.timeunit" to "MILLISECONDS" Do not forget to set related properties. e.g. "org.kie.executor.running.max" to "600000" In case of jBPM 6, you would also need to set "org.kie.executor.interval" to "3000", "org.kie.executor.initial.delay" to "100000"
    • Docs QE Status:
      NEW
    • QE Status:
      NEW

      Description

      When you use ExecutorServiceImpl.requeue(Long olderThan),

      ExecutorServiceImpl converts olderThan (milliseconds) to its timeunit.

      https://github.com/kiegroup/jbpm/blob/7.12.0.Final/jbpm-services/jbpm-executor/src/main/java/org/jbpm/executor/impl/ExecutorServiceImpl.java#L249

      ExecutorRequestAdminServiceImpl expects olderThan is milliseconds.

      https://github.com/kiegroup/jbpm/blob/7.12.0.Final/jbpm-services/jbpm-executor/src/main/java/org/jbpm/executor/impl/jpa/ExecutorRequestAdminServiceImpl.java#L82-L84
      https://github.com/kiegroup/jbpm/blob/7.12.0.Final/jbpm-services/jbpm-executor/src/main/java/org/jbpm/executor/impl/jpa/ExecutorRequestAdminServiceImpl.java#L98-L100

      So if you configure ExecutorServiceImpl.timeunit to "SECONDS", "olderThan" is not a correct value. (e.g. 2000 milliseconds to 2 milliseconds)

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  mmacik Marian Macik
                  Reporter:
                  tkobayashi Toshiya Kobayashi
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: