rymurr commented on a change in pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#discussion_r440906089
##########
File path:
java/memory/src/main/java/org/apache/arrow/memory/rounding/DefaultRoundingPolicy.java
##########
@@ -31,19 +28,18 @@
public final long chunkSize;
+ /**
+ * default chunk size from Netty.
+ */
+ private static final long DEFAULT_CHUNK_SIZE = 16777216L;
Review comment:
It is possible that it changes. It was a bit involved to copy the Netty
code to get chunk size into the arrow codebase so I left it at a likely value
for x86-64 machines. I am happy to replace this with the Netty value or have
chunk size taken from the actual allocator directly if you thikn that is better
than a single static value?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
