Skip to content

[Backport 2.x] Fix compilation error and test failure#228

Merged
heemin32 merged 1 commit into
2.xfrom
backport/backport-210-to-2.x
Jul 6, 2023
Merged

[Backport 2.x] Fix compilation error and test failure#228
heemin32 merged 1 commit into
2.xfrom
backport/backport-210-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport b63c063 from #210

Build failure due to upstream refactoring.

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
(cherry picked from commit b63c063)
@opensearch-trigger-bot opensearch-trigger-bot Bot requested a review from a team February 22, 2023 16:32
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 6, 2023

Codecov Report

Merging #228 (204bacc) into 2.x (b98b5ca) will increase coverage by 0.27%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x     #228      +/-   ##
============================================
+ Coverage     84.98%   85.25%   +0.27%     
+ Complexity      376      356      -20     
============================================
  Files            52       52              
  Lines          1172     1160      -12     
  Branches         97       98       +1     
============================================
- Hits            996      989       -7     
+ Misses          135      133       -2     
+ Partials         41       38       -3     
Impacted Files Coverage Δ
...s/bucket/geogrid/GeoHexGridAggregationBuilder.java 76.00% <ø> (ø)
...ns/bucket/geogrid/GeoHexGridAggregatorFactory.java 100.00% <ø> (ø)
...on/upload/geojson/UploadGeoJSONRequestContent.java 92.68% <100.00%> (+3.02%) ⬆️

... and 11 files with indirect coverage changes

@heemin32 heemin32 merged commit afd673e into 2.x Jul 6, 2023
@gaiksaya gaiksaya deleted the backport/backport-210-to-2.x branch October 17, 2025 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants