[GitHub] [pinot] gortiz commented on pull request #9015: Change Helm chart to log info to console by default

[GitBox <gi...@apache.org>]

gortiz commented on PR #9015:
URL: https://github.com/apache/pinot/pull/9015#issuecomment-1175987741

   > however, should we check against number of lines extra this will create on the k8s logs. INFO is big and not sure if we should do this by default.
   
   I think this is more interesting to people that is just locally trying Pinot. Actual deployments should include their own log4j2.xml or, at least, specify a custom value on this env variable.
   
   Anyway, remember that any k8s deployment should have some log rotation policy configured


-- 
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-unsubscribe@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org