rhodo opened a new pull request, #15628: URL: https://github.com/apache/pinot/pull/15628
This fixes a misconfiguration where `bufAllocatorWithLimits` was set using `.option(ChannelOption.ALLOCATOR, ...)` but not applied via `.childOption(ChannelOption.ALLOCATOR, ...)`. Setting the allocator with .option(...) only affects the parent listening socket, not the child channels (i.e., accepted connections), which are the ones handling actual query traffic and consuming most of the memory. As a result, metrics such as usedDirectMemory were not accurately reflecting the direct memory usage from accepted connections. By applying the allocator to .childOption(...) as well, we ensure that: All connections use the intended allocator with enforced limits Memory metrics now include the buffers allocated for accepted channels -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
