rdblue commented on code in PR #3200:
URL: https://github.com/apache/parquet-java/pull/3200#discussion_r2069317027
##########
parquet-column/src/main/java/org/apache/parquet/schema/PrimitiveStringifier.java:
##########
@@ -442,6 +445,21 @@ private void appendHex(byte[] array, int offset, int
length, StringBuilder build
}
};
+ static final PrimitiveStringifier WKB_STRINGIFIER = new
BinaryStringifierBase("WKB_STRINGIFIER") {
Review Comment:
@wgtmac and @zhangfengcdt, what do you think about updating this to use WKT
only if JTS is available, rather than adding JTS to the classpath for all
downstream clients? This could just produce a generic string if a geo library
isn't available.
--
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]