Add support for S_CONTAINS, S_WITHIN, S_DISJOINT spatial operations

[TravisYeah]

Related Issue(s):

• Support spatial operations WITHIN, CONTAIN, and DISJOINT #371

Description:
Add support for S_CONTAINS, S_WITHIN, S_DISJOINT spatial operations.

PR Checklist:

• Code is formatted and linted (run pre-commit run --all-files)
• Tests pass (run make test)
• Documentation has been updated to reflect changes, if applicable
• Changes are added to the changelog

[jonhealy1]

Hi @TravisYeah, this looks really cool. Can you lint the code so the tests will start. A pre-commit run --all-files. You may need to pre-commit install first. Thank you.

[TravisYeah]

@jonhealy1 thanks for the quick look. Fixed the formatting and the pre-commit passed now.

[jonhealy1]

@TravisYeah These tests pass locally? I am not sure what the issue is immediately ..

[jonhealy1]

Only the within tests are failing?

[jonhealy1]

The polygon dimensions in the test item and the polygon dimensions in the tests for the within operator are exactly the same? Maybe the polygon in the test has to be inside the one in the test item? Try running make test locally if you haven't yet.

[TravisYeah]

@jonhealy1 I was only running the elasticsearch tests and it seems that the opensearch within operator is more strict by default but I fixed the test to work for both now.

[jonhealy1]

@TravisYeah Looks great. I am going to wait to merge this for a couple days just in case anyone else has any comments. Thanks.