use the correct type to widen the sort fields when merging top docs #16881

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

Merged

reta merged 5 commits into opensearch-project:main from bugmakerrrrrr:widen_type

Jan 10, 2025

+152 −23

Codecov / codecov/patch succeeded Jan 9, 2025 in 1s

73.52% of diff hit (target 72.20%)

View this Pull Request on Codecov

73.52% of diff hit (target 72.20%)

Annotations

Check warning on line 628 in server/src/main/java/org/opensearch/action/search/SearchPhaseController.java

codecov / codecov/patch

server/src/main/java/org/opensearch/action/search/SearchPhaseController.java#L627-L628

Added lines #L627 - L628 were not covered by tests

Check warning on line 647 in server/src/main/java/org/opensearch/action/search/SearchPhaseController.java

codecov / codecov/patch

server/src/main/java/org/opensearch/action/search/SearchPhaseController.java#L647

Added line #L647 was not covered by tests

Check warning on line 53 in server/src/main/java/org/opensearch/search/sort/SortedWiderNumericSortField.java

codecov / codecov/patch

server/src/main/java/org/opensearch/search/sort/SortedWiderNumericSortField.java#L53

Added line #L53 was not covered by tests

View more details on Codecov