zhangfengcdt commented on code in PR #2359:
URL: https://github.com/apache/sedona/pull/2359#discussion_r2368799387
##########
spark/common/src/main/java/org/apache/spark/sql/execution/datasources/geoparquet/internal/ParquetColumnVector.java:
##########
@@ -31,11 +33,65 @@
import org.apache.spark.sql.types.DataTypes;
import org.apache.spark.sql.types.MapType;
import org.apache.spark.sql.types.StructType;
+import org.apache.spark.sql.types.UserDefinedType;
/**
* Contains necessary information representing a Parquet column, either of
primitive or nested type.
*/
final class ParquetColumnVector {
Review Comment:
I was trying to see if we can support nested GeometryUDT by make this change
but realized that based on the geoparquet spec, this should not be supported.
The original implementation already reject this so I will revert the changes
here.
--
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]
