Skip to content

Exclude fields from SearchResult#1018

Merged
lucy66hw merged 7 commits intoopensearch-project:mainfrom
karenyrx:unnamed
Jan 2, 2026
Merged

Exclude fields from SearchResult#1018
lucy66hw merged 7 commits intoopensearch-project:mainfrom
karenyrx:unnamed

Conversation

@karenyrx
Copy link
Copy Markdown
Collaborator

@karenyrx karenyrx commented Dec 29, 2025 •
edited
Loading

Description

There is no field named fields in search response

Issues Resolved

Part of opensearch-project/opensearch-protobufs#30

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.

@karenyrx
Exclude unnamed object in SearchResponse from protobufs
4be1956 
Signed-off-by: Karen X <karenxyr@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 29, 2025 •
edited
Loading

Changes Analysis

Commit SHA: 947f3c0
Comparing To SHA: 2e5b859

API Changes

Summary

└─┬Components
  └─┬Schemas
    └─┬_core.search___SearchResult
      └──[➖] properties/fields (51659:9)❌
Document Element Total Changes Breaking Changes
components 1 1
  • ❌ BREAKING Changes: 1 out of 1
  • Removals: 1
  • Breaking Removals: 1

Report

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

API Coverage

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 29, 2025

Spec Test Coverage Analysis

Total Tested
682 680 (99.71 %)

@karenyrx karenyrx marked this pull request as ready for review December 29, 2025 20:13
@karenyrx
remove fields
cbd1e01 
Signed-off-by: Karen X <karenxyr@gmail.com>
@karenyrx
update changelog
a6c6183 
Signed-off-by: Karen X <karenxyr@gmail.com>
@karenyrx
Update CHANGELOG.md
6631e06 
Signed-off-by: Karen X <karenxyr@gmail.com>
@karenyrx
fix typo
4649ea4 
Signed-off-by: Karen X <karenxyr@gmail.com>
@karenyrx
changelog
ed9dad0 
Signed-off-by: Karen X <karenxyr@gmail.com>
@karenyrx karenyrx changed the title Exclude unnamed object in SearchResponse from protobufs Exclude fields from SearchResult Dec 31, 2025
@karenyrx
Merge branch 'main' into unnamed
947f3c0 
Signed-off-by: Karen X <karenxyr@gmail.com>
@lucy66hw lucy66hw merged commit 345a3ae into opensearch-project:main Jan 2, 2026
37 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucy66hw lucy66hw lucy66hw approved these changes

@harshavamsi harshavamsi Awaiting requested review from harshavamsi harshavamsi is a code owner

@sachetalva sachetalva Awaiting requested review from sachetalva sachetalva is a code owner

@Xtansia Xtansia Awaiting requested review from Xtansia Xtansia is a code owner

@VachaShah VachaShah Awaiting requested review from VachaShah VachaShah is a code owner

@Tokesh Tokesh Awaiting requested review from Tokesh Tokesh is a code owner

@aabeshov aabeshov Awaiting requested review from aabeshov aabeshov is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@karenyrx @lucy66hw