Skip to content

[TermsAggregation] update value_type from string to ValueType enum#1057

Merged
karenyrx merged 1 commit intoopensearch-project:mainfrom
yiyuabc:yiyupan/update_value_type_for_terms_aggregation
Feb 25, 2026
Merged

[TermsAggregation] update value_type from string to ValueType enum#1057
karenyrx merged 1 commit intoopensearch-project:mainfrom
yiyuabc:yiyupan/update_value_type_for_terms_aggregation

Conversation

@yiyuabc
Copy link
Copy Markdown
Contributor

@yiyuabc yiyuabc commented Feb 24, 2026

Description

update value_type field type from string to ValueType enum

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@yiyuabc yiyuabc force-pushed the yiyupan/update_value_type_for_terms_aggregation branch from e41ab95 to b104a6c Compare February 24, 2026 23:28
@github-actions
Copy link
Copy Markdown
Contributor

Proto Compatibility Report

Merge Report

No changes detected.


Generated by Proto Compatibility Check

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

Changes Analysis

Commit SHA: b104a6c
Comparing To SHA: 92ea93d

API Changes

Summary

SUCCESS: No changes found between specifications

Report

The full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/22374644410/artifacts/5644805918

API Coverage

Before After Δ
Covered (%) 666 (65.23 %) 666 (65.23 %) 0 (0 %)
Uncovered (%) 355 (34.77 %) 355 (34.77 %) 0 (0 %)
Unknown 149 149 0

@github-actions
Copy link
Copy Markdown
Contributor

Spec Test Coverage Analysis

Total Tested
682 680 (99.71 %)

@karenyrx karenyrx merged commit c88863d into opensearch-project:main Feb 25, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants