mystic-lama commented on code in PR #1743:
URL: https://github.com/apache/orc/pull/1743#discussion_r1449843048
##########
java/core/src/java/org/apache/orc/impl/WriterImpl.java:
##########
@@ -273,6 +277,15 @@ private static int getClosestBufferSize(int size) {
return Math.min(kb256, Math.max(kb4, pow2));
}
+ static {
+ try {
+ com.github.luben.zstd.util.Native.load();
+ } catch (UnsatisfiedLinkError | ExceptionInInitializerError e) {
+ LOG.warn("Unable to load zstd-jni library for your platform... " +
Review Comment:
Minor nit
Can we use single statement and avoid "..."
Unless it violates our checkstyle rules, something like
```
LOG.warn("Unable to load zstd-jni library for your platform. Using
builtin-java classes where applicable");
```
--
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]
