jbonofre opened a new pull request, #1427: URL: https://github.com/apache/arrow-java/pull/1427
## What's Changed `TestNettyAllocator.testMemoryUsage` wrapped a 1024-byte `NO_OP` `ArrowBuf` around the memory address of `PooledByteBufAllocatorL`'s empty buffer, whose native allocation is only a few bytes. The test only reads `memoryAddress()` today, so nothing overruns yet, but any read or write through that buffer would run past the end of the native allocation. That's the same problem #1426 fixes in `TestEmptyArrowBuf`. The buffer isn't needed. The memory usage log the test waits for comes from the status thread that the `PooledByteBufAllocatorL` constructor starts when TRACE logging is enabled, so the test now constructs the allocator directly and drops the `ArrowBuf`. All `memory-netty` tests pass locally on JDK 21 with Error Prone enabled. Closes #238. -- 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]
