sedona icon indicating copy to clipboard operation
sedona copied to clipboard

[SEDONA-138] Fix ST_GeoHash for Flink to work

Open sekikn opened this issue 3 years ago • 1 comments

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

  • Yes, the URL of the assoicated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-138. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?

Currently the ST_GeoHash function for Flink fails with EOFException. This PR fixes it.

How was this patch tested?

By running mvn test -pl flink locally. In addition, I ran the following query from a Flink application:

SELECT ST_GeoHash(ST_Point(0, 1), 5)

then got the following result.

+--------------------------------+
|                         EXPR$0 |
+--------------------------------+
|                          s00j8 |
+--------------------------------+

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the docs.

sekikn avatar Aug 06 '22 14:08 sekikn

To be merged after this PR: https://github.com/apache/incubator-sedona/pull/647

jiayuasu avatar Aug 07 '22 05:08 jiayuasu