kevinrr888 commented on issue #5682:
URL: https://github.com/apache/accumulo/issues/5682#issuecomment-3025378339

   @dlmarion
   > The .audit file being empty is correct. It's defined in 
log4j2-service.properties, but the logger that uses it is commented out in the 
file (on purpose). It exists in the file commented out as an example of what 
the user could enable, but maybe we should also comment out the file appender 
so the file is not created.
   
   Yeah, seems good to not create the file unless the user enables it, if it 
will always be empty otherwise. Did your PR resolve this?
   
   > The .out file being empty is likely correct. It's created in 
accumulo-service and anything that uses System.out should go to this file. 
Nothing should be using System.out. We could remove this file and redirect 
stdout to stderr.
   
   :+1: to the explanation why the .out was empty. I didn't know most of our 
processes don't use stdout at all.
   I don't think we should redirect stdout to stderr, that seems confusing. I 
know at least Admin uses stdout. It doesn't seem like your PR changed this 
though.
   
   > The .err file is created in accumulo-service and there is a log4j2 console 
appender set up to write anything above info to this log. I can change this to 
error in a PR.
   
   :+1:
   
   > The .log file is correct.
   
   :+1:
   
   


-- 
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: notifications-unsubscr...@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to