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.

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

Jump to bottom

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

Merged
merged 5 commits into from
Jun 30, 2023
Merged

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

merged 5 commits into from
Jun 30, 2023

Commits on Jun 28, 2023

  1. Add GeoTile and GeoHash Grid aggregations on GeoShapes. (opensearch-p… 

    …roject#5589)

Src files for GeoTile and GeoHash Aggregations on GeoShape with integration
tests.

Signed-off-by: Navneet Verma <[email protected]>
    @navneet1v @heemin32
    navneet1v authored and heemin32 committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    b39f18b View commit details
    Browse the repository at this point in the history

  2. [opensearch-project/geospatial#212] Fixing the IT for GeoTilesAggrega… ( 

    opensearch-project#6120)

Fixing the IT for GeoTilesAggregation.

Signed-off-by: Navneet Verma <[email protected]>
    @navneet1v @heemin32
    navneet1v authored and heemin32 committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    a66c3ba View commit details
    Browse the repository at this point in the history

  3. [opensearch-project#6187, opensearch-project#6222] Fixing the GeoShap… 

    …es GeoHash and GeoTile Aggregations Integration tests. (opensearch-project#6242)

Changes done:
* Fixed the ArrayIndexOutOfBoundsException.
* Reduced the precision for GeoShapes Aggregation IT testing.

Signed-off-by: Navneet Verma <[email protected]>
    @navneet1v @heemin32
    navneet1v authored and heemin32 committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    9105f0c View commit details
    Browse the repository at this point in the history

  4. [opensearch-project#7101] Fixing the GeoTileIT#testMultivaluedGeoPoin… 

    …tsAggregation test case. (opensearch-project#7166)

The issue was happening because we encode the GeoPoint as long and error comes in the precision due to that encoding. The error was not taken care while generating the exepected tiles count for execpected output.

Signed-off-by: Navneet Verma <[email protected]>
    @navneet1v @heemin32
    navneet1v authored and heemin32 committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    9d1fa92 View commit details
    Browse the repository at this point in the history

Commits on Jun 29, 2023

  1. Merge branch '2.x' into agg 

    Signed-off-by: Heemin Kim <[email protected]>
    @heemin32
    heemin32 authored Jun 29, 2023
    Configuration menu
    Copy the full SHA
    8826015 View commit details
    Browse the repository at this point in the history