ZhuochengShang commented on code in PR #2221:
URL: https://github.com/apache/sedona/pull/2221#discussion_r2252708460
##########
common/src/main/java/org/apache/sedona/common/S2Geography/SinglePointGeography.java:
##########
@@ -22,7 +22,7 @@
public class SinglePointGeography extends PointGeography {
public SinglePointGeography(S2Point p) {
- super(p);
+ super(GeographyKind.SINGLEPOINT, p);
Review Comment:
PointGeography accepts a list of S2Point instances, while
SinglePointGeography must contain exactly one S2Point. Here we added two new
GeographyKind: SINGLEPOINT and SINGLEPOLYLINE. The reason is, previously, with
only POINT defined, both WKT “POINT” and “MULTIPOINT” were serialized as
“MULTIPOINT.” By introducing these more specific kinds, we can output correct
WKT format string.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
