[Hawkular-commits] [hawkular/hawkular-metrics] 27eae4: [HWKMETRICS-564] fix several bugs

[Stefan Negrea]

  Branch: refs/heads/release/0.23.0
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: 27eae4dcfa0bfabae2bdb3e53ef6f5ab55b75f0e
      https://github.com/hawkular/hawkular-metrics/commit/27eae4dcfa0bfabae2bdb3e53ef6f5ab55b75f0e
  Author: John Sanda <jsanda at redhat.com>
  Date:   2017-01-11 (Wed, 11 Jan 2017)

  Changed paths:
    M job-scheduler/src/main/java/org/hawkular/metrics/scheduler/impl/JobsService.java
    M job-scheduler/src/main/java/org/hawkular/metrics/scheduler/impl/SchedulerImpl.java

  Log Message:
  -----------
  [HWKMETRICS-564] fix several bugs

When jobs fall behind and are rescheduled for immediate execution, we first
renew the job lock. The job should only be executed again if the lock is
successfully renewed. We were always executing the job. This is fixed.

When there are jobs to execute across multiple time slices, we were executing
things out of order. This is fixed simply by replacing flatMap with concatMap
in a few places.

When retrying a job execution, we were updating finished_jobs_idx with the
wrong timestamp. We were using the trigger time instead of the time in which
the job is scheduled. In this scenario, the scheduled time != the trigger time.


  Commit: 17c1288b9e32bf9be76f5bc955a85b9a79023464
      https://github.com/hawkular/hawkular-metrics/commit/17c1288b9e32bf9be76f5bc955a85b9a79023464
  Author: John Sanda <jsanda at redhat.com>
  Date:   2017-01-11 (Wed, 11 Jan 2017)

  Changed paths:
    M job-scheduler/src/main/java/org/hawkular/metrics/scheduler/impl/SchedulerImpl.java

  Log Message:
  -----------
  [HWKMETRICS-564] changes based on PR review


  Commit: 12edcfa57d70fe1155cfc5abacd4b87cc8c7573f
      https://github.com/hawkular/hawkular-metrics/commit/12edcfa57d70fe1155cfc5abacd4b87cc8c7573f
  Author: Stefan Negrea <stefannegrea at users.noreply.github.com>
  Date:   2017-01-11 (Wed, 11 Jan 2017)

  Changed paths:
    M job-scheduler/src/main/java/org/hawkular/metrics/scheduler/impl/JobsService.java
    M job-scheduler/src/main/java/org/hawkular/metrics/scheduler/impl/SchedulerImpl.java

  Log Message:
  -----------
  Merge pull request #724 from jsanda/release/0.23.0-hwkmetrics-564

HWKMETRICS-564


Compare: https://github.com/hawkular/hawkular-metrics/compare/e6f351ae7683...12edcfa57d70