Copilot commented on code in PR #2649: URL: https://github.com/apache/sedona/pull/2649#discussion_r2807111067
########## spark/common/src/main/scala/org/apache/spark/sql/sedona_sql/expressions/Functions.scala: ########## @@ -32,6 +32,7 @@ import org.apache.spark.sql.sedona_sql.expressions.implicits._ import org.apache.spark.sql.types._ import org.locationtech.jts.algorithm.MinimumBoundingCircle import org.locationtech.jts.geom._ +import org.locationtech.jts.geom.Geometry Review Comment: The explicit import of `org.locationtech.jts.geom.Geometry` is added on line 35 but there's already a wildcard import of `org.locationtech.jts.geom._` on line 34. The explicit import is redundant since the wildcard import already includes Geometry. While this doesn't cause compilation errors, it's unnecessary code duplication that could be simplified by removing the explicit import on line 35. ```suggestion ``` -- 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]
