epugh commented on PR #894: URL: https://github.com/apache/solr/pull/894#issuecomment-1147622724
> I'm not sure I understand why this change is needed. If "remote" is the default in jaeger, and "remote" requires gson - why are we excluding it? It just seems like the documentation was incorrect about needing to manually add gson. I don't think we should be excluding dependencies on a regular basis - especially if they are needed for things to work by default. @dsmiley can you weigh in on this? -- 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