cpoerschke commented on PR #927: URL: https://github.com/apache/solr/pull/927#issuecomment-1179084190
> > Yea, I don't have strong feelings about leaving the loggers in over taking them out, do what you gotta do man. > > @cpoerschke want to be tie breaker? No strong views either way. As a developer I find it convenient if the logger is already there but from a CI perspective I'd also love to see the tools flagging up early when code becomes unused in future. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
