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.

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

[Backport 2.x] GeoTile, and GeoGrid aggregation #8295

Merged
merged 5 commits into from
Jun 30, 2023

Conversation

heemin32
Copy link
Contributor

@heemin32 heemin32 commented Jun 27, 2023

Description

Back porting of GeoTile, and GeoGrid aggregation to 2.x

Related Issues

Resolves

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Jun 27, 2023
@heemin32 heemin32 reopened this Jun 27, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #8295 (8826015) into 2.x (b12a45a) will increase coverage by 0.02%.
The diff coverage is 18.26%.

@@             Coverage Diff              @@
##                2.x    #8295      +/-   ##
============================================
+ Coverage     70.62%   70.64%   +0.02%     
+ Complexity    56924    56906      -18     
============================================
  Files          4714     4717       +3     
  Lines        269648   269749     +101     
  Branches      39857    39873      +16     
============================================
+ Hits         190434   190572     +138     
+ Misses        62857    62776      -81     
- Partials      16357    16401      +44     
Impacted Files Coverage Δ
.../main/java/org/opensearch/geo/GeoModulePlugin.java 100.00% <ø> (ø)
...cket/composite/GeoTileGridValuesSourceBuilder.java 89.65% <ø> (ø)
.../bucket/geogrid/GeoHashGridAggregationBuilder.java 100.00% <ø> (ø)
.../bucket/geogrid/GeoTileGridAggregationBuilder.java 100.00% <ø> (ø)
...ations/bucket/geogrid/cells/BoundedCellValues.java 100.00% <ø> (ø)
...ggregations/bucket/geogrid/cells/CellIdSource.java 64.28% <ø> (ø)
.../aggregations/bucket/geogrid/cells/CellValues.java 100.00% <ø> (ø)
...ons/bucket/geogrid/cells/GeoShapeCellIdSource.java 0.00% <0.00%> (ø)
...tions/bucket/geogrid/cells/GeoShapeCellValues.java 0.00% <0.00%> (ø)
...ions/bucket/geogrid/cells/UnboundedCellValues.java 100.00% <ø> (ø)
... and 13 more

... and 468 files with indirect coverage changes

@heemin32 heemin32 closed this Jun 28, 2023
@heemin32 heemin32 reopened this Jun 28, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Jun 28, 2023
@heemin32 heemin32 reopened this Jun 28, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Jun 28, 2023
@heemin32 heemin32 reopened this Jun 28, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.remotestore.SegmentReplicationUsingRemoteStoreIT.testPressureServiceStats
      1 org.opensearch.remotestore.SegmentReplicationUsingRemoteStoreIT.testDropPrimaryDuringReplication

@andrross andrross added feature New feature or request backport PRs or issues specific to backporting features or enhancments Geospatial v2.9.0 'Issues and PRs related to version v2.9.0' labels Jun 29, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Jun 29, 2023
@heemin32 heemin32 reopened this Jun 29, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Jun 30, 2023
@heemin32 heemin32 reopened this Jun 30, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Jun 30, 2023
@heemin32 heemin32 reopened this Jun 30, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@andrross andrross merged commit 0dbafea into opensearch-project:2.x Jun 30, 2023
@heemin32 heemin32 deleted the agg branch June 30, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport PRs or issues specific to backporting features or enhancments feature New feature or request Geospatial v2.9.0 'Issues and PRs related to version v2.9.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants