[Hawkular-commits] [hawkular/hawkular-metrics] 915874: HWKMETRICS-390 Endpoint for fetching rate data poi...

Branch: refs/heads/master Home: https://github.com/hawkular/hawkular-metrics Commit: 915874767369d655c66deb86f8c4c8532063e128 https://github.com/hawkular/hawkular-metrics/commit/915874767369d655c66de... Author: Thomas Segismont <tsegismo(a)redhat.com&gt; Date: 2016-05-03 (Tue, 03 May 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 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 R core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/NumericDataPointCollector.java M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/Order.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/PatternUtil.java R core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/TaggedDataPointCollector.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/NumericBucketPointTransformer.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/NumericDataPointCollector.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/TaggedBucketPointTransformer.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/TaggedDataPointCollector.java M core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/GenerateRateITest.java M core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/MetricsServiceITest.java M core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/RatesITest.java M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy Log Message: ----------- HWKMETRICS-390 Endpoint for fetching rate data points need to support standard query params and sort behavior I added a method in Order.java which factors out the logic to determine a default order. I updated the considering endpoints. I change the get rate endpoint to support the new parameters. While its bucket parameters are deprecated and not removed, we need to check that they can't be set if order and limit are. While updating the code in MetricsServiceImpl to adapt to the new findRate signature, I extracted the code of #bucketize methods and created separated Transformer classes. Using these transformers with #compose is a more natural RxJava construct. Commit: 1dc367ee78512dd859b6927e0c5db7eb630a9430 https://github.com/hawkular/hawkular-metrics/commit/1dc367ee78512dd859b69... Author: Thomas Segismont <tsegismo(a)redhat.com&gt; Date: 2016-05-03 (Tue, 03 May 2016) Changed paths: M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/TaggedBucketPointTransformer.java Log Message: ----------- Refactored TaggedBucketPointTransformer to be more functional Create a Predicate<DataPoint> with Predicate#and instead of a list of functions. Then when building the Observable we only have to call Predicate#test Commit: baea70154e19e01c8becc0884b4c5da4fd3ad68e https://github.com/hawkular/hawkular-metrics/commit/baea70154e19e01c8becc... Author: jsanda <jsanda(a)redhat.com&gt; Date: 2016-05-03 (Tue, 03 May 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 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 R core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/NumericDataPointCollector.java M core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/Order.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/PatternUtil.java R core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/TaggedDataPointCollector.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/NumericBucketPointTransformer.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/NumericDataPointCollector.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/TaggedBucketPointTransformer.java A core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/transformers/TaggedDataPointCollector.java M core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/GenerateRateITest.java M core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/MetricsServiceITest.java M core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/RatesITest.java M integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy Log Message: ----------- Merge pull request #497 from tsegismont/jira/HWKMETRICS-390 HWKMETRICS-390 Endpoint for fetching rate data points need to support standard query params and sort behavior Compare: https://github.com/hawkular/hawkular-metrics/compare/937d9c6e5672...baea7...