Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Metric Framework] Adds support for Histogram metric #12062

Merged
merged 11 commits into from
Feb 5, 2024

Addresses review comments

a0b5034
Select commit
Loading
Failed to load commit list.
Merged

[Metric Framework] Adds support for Histogram metric #12062

Addresses review comments
a0b5034
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Feb 5, 2024 in 1s

70.58% of diff hit (target 71.26%)

View this Pull Request on Codecov

70.58% of diff hit (target 71.26%)

Annotations

Check warning on line 25 in libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopHistogram.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopHistogram.java#L25 

Added line #L25 was not covered by tests

Check warning on line 32 in libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopHistogram.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopHistogram.java#L32 

Added line #L32 was not covered by tests

Check warning on line 37 in libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopHistogram.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopHistogram.java#L37 

Added line #L37 was not covered by tests

Check warning on line 44 in libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopMetricsRegistry.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

libs/telemetry/src/main/java/org/opensearch/telemetry/metrics/noop/NoopMetricsRegistry.java#L44 

Added line #L44 was not covered by tests

Check warning on line 1851 in server/src/main/java/org/opensearch/index/IndexSettings.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/IndexSettings.java#L1850-L1851 

Added lines #L1850 - L1851 were not covered by tests

Check warning on line 1860 in server/src/main/java/org/opensearch/index/IndexSettings.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/IndexSettings.java#L1858-L1860 

Added lines #L1858 - L1860 were not covered by tests

Check warning on line 1862 in server/src/main/java/org/opensearch/index/IndexSettings.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/IndexSettings.java#L1862 

Added line #L1862 was not covered by tests

Check warning on line 1866 in server/src/main/java/org/opensearch/index/IndexSettings.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/IndexSettings.java#L1866 

Added line #L1866 was not covered by tests

Check warning on line 1868 in server/src/main/java/org/opensearch/index/IndexSettings.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/IndexSettings.java#L1868 

Added line #L1868 was not covered by tests

Check warning on line 1875 in server/src/main/java/org/opensearch/index/IndexSettings.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/IndexSettings.java#L1875 

Added line #L1875 was not covered by tests

Check warning on line 110 in server/src/main/java/org/opensearch/index/codec/fuzzy/BloomFilter.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/BloomFilter.java#L110 

Added line #L110 was not covered by tests

Check warning on line 137 in server/src/main/java/org/opensearch/index/codec/fuzzy/BloomFilter.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/BloomFilter.java#L137 

Added line #L137 was not covered by tests

Check warning on line 92 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L91-L92 

Added lines #L91 - L92 were not covered by tests

Check warning on line 148 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L148 

Added line #L148 was not covered by tests

Check warning on line 155 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L155 

Added line #L155 was not covered by tests

Check warning on line 181 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L181 

Added line #L181 was not covered by tests

Check warning on line 268 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L266-L268 

Added lines #L266 - L268 were not covered by tests

Check warning on line 294 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L294 

Added line #L294 was not covered by tests

Check warning on line 307 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L306-L307 

Added lines #L306 - L307 were not covered by tests

Check warning on line 316 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L316 

Added line #L316 was not covered by tests

Check warning on line 341 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L341 

Added line #L341 was not covered by tests

Check warning on line 352 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L352 

Added line #L352 was not covered by tests

Check warning on line 386 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L386 

Added line #L386 was not covered by tests

Check warning on line 415 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L414-L415 

Added lines #L414 - L415 were not covered by tests

Check warning on line 490 in server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/codec/fuzzy/FuzzyFilterPostingsFormat.java#L490 

Added line #L490 was not covered by tests
View more details on Codecov