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

[FlakyTest]org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitorTests.testIndexingMultiPolygon #294

Open
heemin32 opened this issue May 5, 2023 · 2 comments
Labels

Comments

@heemin32
Copy link
Collaborator

heemin32 commented May 5, 2023

REPRODUCE WITH: ./gradlew ':test' --tests "org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitorTests.testIndexingMultiPolygon" -Dtests.seed=C317A78A633C1A58 -Dtests.security.manager=false -Dtests.locale=ar-TN -Dtests.timezone=Australia/Darwin -Druntime.java=17
> Task :test

org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitorTests > testIndexingMultiPolygon FAILED
    java.lang.IllegalArgumentException: at least three non-collinear points required
        at __randomizedtesting.SeedInfo.seed([C317A78A633C1A58:FF568A98481069AA]:0)

        at org.apache.lucene.geo.Tessellator.tessellate(Tessellator.java:174)

        at org.apache.lucene.geo.Tessellator.tessellate(Tessellator.java:152)
Suite: Test class org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitorTests
  2> REPRODUCE WITH: ./gradlew ':test' --tests "org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitorTests.testIndexingMultiPolygon" -Dtests.seed=C317A78A633C1A58 -Dtests.security.manager=false -Dtests.locale=ar-TN -Dtests.timezone=Australia/Darwin -Druntime.java=17
  2> java.lang.IllegalArgumentException: at least three non-collinear points required
        at __randomizedtesting.SeedInfo.seed([C317A78A633C1A58:FF568A98481069AA]:0)
        at org.apache.lucene.geo.Tessellator.tessellate(Tessellator.java:174)
        at org.apache.lucene.geo.Tessellator.tessellate(Tessellator.java:152)
        at org.apache.lucene.document.XYShape.createIndexableFields(XYShape.java:100)
        at org.apache.lucene.document.XYShape.createIndexableFields(XYShape.java:84)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.createIndexableFields(XYShapeIndexableFieldsVisitor.java:107)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.visit(XYShapeIndexableFieldsVisitor.java:97)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.visit(XYShapeIndexableFieldsVisitor.java:40)
        at org.opensearch.geometry.Polygon.visit(Polygon.java:109)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.lambda$visitCollection$0(XYShapeIndexableFieldsVisitor.java:112)
        at java.base/java.lang.Iterable.forEach(Iterable.java:75)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.visitCollection(XYShapeIndexableFieldsVisitor.java:112)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.visit(XYShapeIndexableFieldsVisitor.java:84)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitor.visit(XYShapeIndexableFieldsVisitor.java:40)
        at org.opensearch.geospatial.index.mapper.xyshape.XYShapeIndexableFieldsVisitorTests.testIndexingMultiPolygon(XYShapeIndexableFieldsVisitorTests.java:121)
@heemin32 heemin32 added bug Something isn't working untriaged flaky test and removed untriaged bug Something isn't working labels May 5, 2023
heemin32 added a commit to heemin32/geospatial that referenced this issue May 10, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit to heemin32/geospatial that referenced this issue May 10, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit to heemin32/geospatial that referenced this issue May 10, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit to heemin32/geospatial that referenced this issue May 11, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit to heemin32/geospatial that referenced this issue May 11, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit to heemin32/geospatial that referenced this issue May 11, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
@heemin32
Copy link
Collaborator Author

There is a bug in this method https://github.com/apache/lucene/blob/main/lucene/test-framework/src/java/org/apache/lucene/tests/geo/ShapeTestUtil.java#L156

It fails to generate correct polygon.

heemin32 added a commit to heemin32/geospatial that referenced this issue Sep 26, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
@heemin32 heemin32 added the good first issue Good for newcomers label Sep 26, 2023
heemin32 added a commit to heemin32/geospatial that referenced this issue Sep 26, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
@heemin32
Copy link
Collaborator Author

The issue is temporary fixed as of now. For a proper fix, we need a fix in lucene library.
apache/lucene#12287 and apache/lucene#12596

heemin32 added a commit to heemin32/geospatial that referenced this issue Sep 26, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit to heemin32/geospatial that referenced this issue Sep 26, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
heemin32 added a commit that referenced this issue Sep 26, 2023
Signed-off-by: Heemin Kim <heemin@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant