cshannon commented on issue #3094:
URL: https://github.com/apache/accumulo/issues/3094#issuecomment-1335503578

   https://github.com/apache/accumulo/pull/3103 addresses the null logging 
issue from the original patch in #3047 
   
   The problem is the server is nonblocking so different threads can process 
the same FrameBuffer. When I tested and increased logging I saw different 
threads calling read() and invoke() on the same buffer so capturing the client 
address in invoke() doesn't work. The new PR just captures the client address 
on FrameBuffer creation and stores it in a String format for easy logging as a 
new FrameBuffer is allocated for each request. 
   
   With this new patch the logging should now work correctly and not return 
null.


-- 
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]

Reply via email to