7:33 p.m.
Branch: refs/heads/master
Home: https://github.com/hawkular/hawkular-metrics
Commit: 4cdd6be407faa0b7a467de0b9ae7f4c7682c24d2
https://github.com/hawkular/hawkular-metrics/commit/4cdd6be407faa0b7a467d...
Author: John Sanda <jsanda(a)redhat.com>
Date: 2018-05-04 (Fri, 04 May 2018)
Changed paths:
M
core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/MetricsServiceImpl.java
M
core/metrics-core-service/src/main/java/org/hawkular/metrics/core/service/tags/SimpleTagQueryParser.java
M
core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/metrics/TagsITest.java
M
core/metrics-core-service/src/test/java/org/hawkular/metrics/core/service/tags/SimpleTagQueryParserTest.java
M
integration-tests/rest-tests-jaxrs/src/test/groovy/org/hawkular/metrics/rest/TenantITest.groovy
Log Message:
-----------
[HWKMETRICS-787] optimize for pod_id queries (#961)
* Optimize Openshift tag queries
* [HWKMETRICS-787] clean up pod_id tag query logic and add tests
* [HWKMETRICS-787] clean up pod_id query and add some debug logging
I removed the reduce operation since it is not needed. This whole
optimization is for a particular tag key. There can only be one instance
of a tag key per request; hence, the reduce operation is not needed.
* [HWKMETRICS-787] disable failing test
I spent a few minutes investigating, but I am not going to wast any more
time. The delete tenant functionality has never been used, and the REST
integration tests are tricky due to using the "virtual clock" which in
turn makes these tests brittle.
I will open a separate ticket for disabling the delete tenant
functionality. We can leave the endpoints intact but return an
appropriate status and error message indicating that it is no longer
supported.
**NOTE:** This service been marked for deprecation:
https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.