[Hawkular-commits] [hawkular/hawkular-metrics] 384cec: [HWKMETRICS-393] add endpoint for fetching raw dat...

Wed Jun 22 10:53:00 EDT 2016

  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: 384cec2b01cf02de21e070999edc30c7828918e8
      https://github.com/hawkular/hawkular-metrics/commit/384cec2b01cf02de21e070999edc30c7828918e8
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/QueryRequest.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/observer/NamedDataPointObserver.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/JacksonConfig.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/ObjectMapperProducer.java
    M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsService.java
    M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsServiceImpl.java
    A core/metrics-model/src/main/java/org/hawkular/metrics/model/NamedDataPoint.java
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/GaugesITest.groovy

  Log Message:
  -----------
  [HWKMETRICS-393] add endpoint for fetching raw data from multiple gauges

This new endpoint goes against REST design principles by using POST for a
query. This has been done in order to avoid problems with complex and long
query strings in the URL.

This endpoint is also a bit different in that it streams the resposne. The
reason for this is that the client can request an arbitrary amount of data
from an arbitrary number of metrics.

  Commit: b4253c3865400ecd9e6ea9558507c99e120bf405
      https://github.com/hawkular/hawkular-metrics/commit/b4253c3865400ecd9e6ea9558507c99e120bf405
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/AvailabilityHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/AvailabilityITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/GaugesITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/RESTTest.groovy

  Log Message:
  -----------
  [HWKMETRICS-393] add endpoint for counters and availability

This commit adds endpoint for fetch raw data from multiple counters and
availability metrics.

  Commit: 5b72109eba8e5c603003b04c11811073e423ef52
      https://github.com/hawkular/hawkular-metrics/commit/5b72109eba8e5c603003b04c11811073e423ef52
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/AvailabilityITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/StringITest.groovy

  Log Message:
  -----------
  [HWKMETRICS-393] add endpoint for fetching raw data from multiple string metrics

  Commit: d75ff365ef32b06e94d3549c95aff041fe946326
      https://github.com/hawkular/hawkular-metrics/commit/d75ff365ef32b06e94d3549c95aff041fe946326
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java
    M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsService.java
    M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsServiceImpl.java
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/GaugesITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/RESTTest.groovy

  Log Message:
  -----------
  [HWKMETRICS-393] add endpoints for fetching rate data points from multiple metrics

  Commit: 09471a2f13886372be16585d5ade8163498270bf
      https://github.com/hawkular/hawkular-metrics/commit/09471a2f13886372be16585d5ade8163498270bf
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/QueryRequest.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/observer/NamedDataPointObserver.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/ObjectMapperProducer.java
    M core/metrics-model/src/main/java/org/hawkular/metrics/model/NamedDataPoint.java

  Log Message:
  -----------
  [HWKMETRICS-393] fix license and checkstyle errors

  Commit: 3efbb9c849d913f52f25642c80d01a1be7f3e462
      https://github.com/hawkular/hawkular-metrics/commit/3efbb9c849d913f52f25642c80d01a1be7f3e462
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy

  Log Message:
  -----------
  [HWKMETRICS-393] relax verification since order of results may change

  Commit: c6009b40e7ac8385eb311d49bafdcf494114278b
      https://github.com/hawkular/hawkular-metrics/commit/c6009b40e7ac8385eb311d49bafdcf494114278b
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/AvailabilityHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricsServiceHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/observer/NamedDataPointObserver.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/ObjectMapperProducer.java

  Log Message:
  -----------
  [HWKMERTICS-393] several changes based on PR review

I eliminated most of the duplicate code by pushing it up into a new parent
class, MetricsServiceHandler. The name sucks, but both BaseHandler and
MetricHandler are already taken.

I have added a bunch of swagger annotations for the new endpoints.

Lastly, I put the CountDownLatch.countDown() call in a finally block to ensure
it happens to prevent blocking indefinitely.

  Commit: b556b848291808a4af2fbddef436ad4b1d947128
      https://github.com/hawkular/hawkular-metrics/commit/b556b848291808a4af2fbddef436ad4b1d947128
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricsServiceHandler.java

  Log Message:
  -----------
  [HWKMETRICS-393] fix license header

  Commit: b4ddfc05e17045d6df8781987c1cb925551c05f3
      https://github.com/hawkular/hawkular-metrics/commit/b4ddfc05e17045d6df8781987c1cb925551c05f3
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java

  Log Message:
  -----------
  [HWKMETRICS-393] fix checkstyle/license errors

  Commit: e559e28482f51dd30805b63da09efce71538490b
      https://github.com/hawkular/hawkular-metrics/commit/e559e28482f51dd30805b63da09efce71538490b
  Author: John Sanda <jsanda at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/AvailabilityHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java

  Log Message:
  -----------
  [HWKMETRICS-393] update swagger docs to indicate these endpoints are experimental

  Commit: 01122e0a8711e917430b640195f4a347fec2ba2e
      https://github.com/hawkular/hawkular-metrics/commit/01122e0a8711e917430b640195f4a347fec2ba2e
  Author: Stefan Negrea <snegrea at redhat.com>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/QueryRequest.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/AvailabilityHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricsServiceHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/observer/NamedDataPointObserver.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/JacksonConfig.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/ObjectMapperProducer.java
    M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsService.java
    M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsServiceImpl.java
    A core/metrics-model/src/main/java/org/hawkular/metrics/model/NamedDataPoint.java
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/AvailabilityITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/GaugesITest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/RESTTest.groovy
    M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/StringITest.groovy

  Log Message:
  -----------
  Merge pull request #518 from jsanda/hwkmetrics-393

HWKMETRICS-393

Compare: https://github.com/hawkular/hawkular-metrics/compare/2283942e5dba...01122e0a8711