szehon-ho commented on PR #53227: URL: https://github.com/apache/spark/pull/53227#issuecomment-3667628847
had a chat with @uros-db and @mkaravel , the first patch made subclasses of Geometry (like Point, Line, etc) and made it public. But actually it is to be used only internally by the WkbReader and WkbWriter, and should not be exposed (at least for now). The logic of Geometry.fromWKB is simply to call WKBReader to parse and validate the wkb, before setting the simple internal representation (srid ++ wkb), and Geometry.toWKB can call WkbReader + WkbWriter to write the internal (srid ++ wkb) to the right endianness. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
