epugh commented on PR #3910: URL: https://github.com/apache/solr/pull/3910#issuecomment-3593679504
> Please not yet another bats test; they are very heavy, and only a couple people maintain them. > > Its weird that solrconfig.xml has an infoStream setting. I suppose to prevent accidental enabling of very low level logging. But IMO that should be addressed with suitable log levels. This should be debug or trace level, and furthermore use a Lucene namespace not Solr, which is merely adding a factory in a sense. Then we wouldn't feel the need for a separate infoStream setting. There is a lot I might want to do to improve LoggingInfoStream, albeit out-of-scope here. > > Another problem is that it's misleading looking at this log configuration. One would _easily_ guess that they will see this low level stuff. That's a trap. Eric, can you please add a comment that the logging is actually/mostly controlled by solrconfig.xml infoStream ? I think one reason that infostream isn't controlled by log4j2.xml is because you can choose PER core (collection) if it should be logging, and that's why it's in solrconfig.xml. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
