[Hawkular-commits] [hawkular/hawkular-metrics] 5e88a3: HWKMETRICS-154 List metrics endpoint fails with 50...

Michael Burman yak at iki.fi
Tue Jun 23 08:06:24 EDT 2015


  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: 5e88a31e86fbe2ee0fe7c224a6eb68c5885fd631
      https://github.com/hawkular/hawkular-metrics/commit/5e88a31e86fbe2ee0fe7c224a6eb68c5885fd631
  Author: Thomas Segismont <tsegismo at redhat.com>
  Date:   2015-06-23 (Tue, 23 Jun 2015)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/param/ConvertersProvider.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/param/MetricTypeConverter.java
    A api/metrics-api-jaxrs/src/test/java/org/hawkular/metrics/api/jaxrs/param/MetricTypeConverterTest.java

  Log Message:
  -----------
  HWKMETRICS-154 List metrics endpoint fails with 500 Internal Server Error when "type" param is missing

Fixed MetricHandler to make sure the param is not null.

Also, MetricType now has a JAX-RS converter which makes the code cleaner.


  Commit: 464e67247cbf340a00944ba2240a7ba206f2cc62
      https://github.com/hawkular/hawkular-metrics/commit/464e67247cbf340a00944ba2240a7ba206f2cc62
  Author: Thomas Segismont <tsegismo at redhat.com>
  Date:   2015-06-23 (Tue, 23 Jun 2015)

  Changed paths:
    M core/metrics-core-api/src/main/java/org/hawkular/metrics/core/api/MetricType.java

  Log Message:
  -----------
  MetricType refactoring.

* fixed Javadoc
* use maps for lookups instead of switch (less error prone because you just need to declare the new value when adding a type)


  Commit: 8259a53dbf2cd0d8cbcd39e8096f5af0c398ba9c
      https://github.com/hawkular/hawkular-metrics/commit/8259a53dbf2cd0d8cbcd39e8096f5af0c398ba9c
  Author: Thomas Segismont <tsegismo at redhat.com>
  Date:   2015-06-23 (Tue, 23 Jun 2015)

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

  Log Message:
  -----------
  No ApiError body for 20X responses


  Commit: 14d9c1a567c2dfaf7941539acdc130ba2248ea13
      https://github.com/hawkular/hawkular-metrics/commit/14d9c1a567c2dfaf7941539acdc130ba2248ea13
  Author: Thomas Segismont <tsegismo at redhat.com>
  Date:   2015-06-23 (Tue, 23 Jun 2015)

  Changed paths:
    M core/metrics-core-api/src/main/java/org/hawkular/metrics/core/api/MetricType.java

  Log Message:
  -----------
  Add unique code/text checks


  Commit: ca69df29782d54626688426609a674657ef1d8e7
      https://github.com/hawkular/hawkular-metrics/commit/ca69df29782d54626688426609a674657ef1d8e7
  Author: Thomas Segismont <tsegismo at redhat.com>
  Date:   2015-06-23 (Tue, 23 Jun 2015)

  Changed paths:
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/CassandraBackendITest.groovy

  Log Message:
  -----------
  HWKMETRICS-155 Test against regression for metric definition list status code 400


  Commit: ef2a653fd9a7529bd3a004aec16f2670b8ff1650
      https://github.com/hawkular/hawkular-metrics/commit/ef2a653fd9a7529bd3a004aec16f2670b8ff1650
  Author: Michael Burman <yak at iki.fi>
  Date:   2015-06-23 (Tue, 23 Jun 2015)

  Changed paths:
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricHandler.java
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/param/ConvertersProvider.java
    A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/param/MetricTypeConverter.java
    A api/metrics-api-jaxrs/src/test/java/org/hawkular/metrics/api/jaxrs/param/MetricTypeConverterTest.java
    M core/metrics-core-api/src/main/java/org/hawkular/metrics/core/api/MetricType.java
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/CassandraBackendITest.groovy

  Log Message:
  -----------
  Merge pull request #270 from tsegismont/jira/HWKMETRICS-154

List metrics endpoint fails with 500 Internal Server Error when "type" param is missing


Compare: https://github.com/hawkular/hawkular-metrics/compare/c8af239d2e82...ef2a653fd9a7


More information about the hawkular-commits mailing list