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

[Stefan Negrea]

  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: eacccd7ff3aec11c306efbb3977da338218284bc
      https://github.com/hawkular/hawkular-metrics/commit/eacccd7ff3aec11c306efbb3977da338218284bc
  Author: John Sanda <jsanda at redhat.com>
  Date:   2017-01-06 (Fri, 06 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: 91cb9007a400b356c03dced348d6fd9cfdeb3904
      https://github.com/hawkular/hawkular-metrics/commit/91cb9007a400b356c03dced348d6fd9cfdeb3904
  Author: John Sanda <jsanda at redhat.com>
  Date:   2017-01-09 (Mon, 09 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: 028528516fb315f232874d0de1e6301a1ca3d53e
      https://github.com/hawkular/hawkular-metrics/commit/028528516fb315f232874d0de1e6301a1ca3d53e
  Author: Stefan Negrea <stefannegrea at users.noreply.github.com>
  Date:   2017-01-10 (Tue, 10 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 #719 from jsanda/hwkmetrics-564

[HWKMETRICS-564] fix several bugs


Compare: https://github.com/hawkular/hawkular-metrics/compare/8489455cfa03...028528516fb3