Kontinuation commented on code in PR #1811:
URL: https://github.com/apache/sedona/pull/1811#discussion_r1962701176
##########
common/src/main/java/org/apache/sedona/common/Functions.java:
##########
@@ -784,6 +785,10 @@ public static byte[] asEWKB(Geometry geometry) {
return GeomUtils.getEWKB(geometry);
}
+ public static byte[] geogAsEWKB(Geography geography) {
Review Comment:
There's a (quite complex syntax) for referring to a specific overload of
functions in Scala. Even if we support argument type based overloading in
Inferred expression, we still have to list all overloads we want to delegate
to. For instance
```scala
InferredExpression(
(g: Geography) => Functions.asEWKB(g),
(g: Geometry) => Functions.asEWKB(g))
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@sedona.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
