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]

Reply via email to