epugh opened a new pull request, #4123: URL: https://github.com/apache/solr/pull/4123
# Description This is a follow up to https://github.com/apache/solr/pull/4073 where we discovered we need "raw" response writers to be at the node/container level. # Solution Add "raw" to the `ResponseWritersRegistery`. Now I see why we wish handlers registered their own response writers with SolrCore or the ResponseWritersRegistry. Too bad we don't have a Dependency Injection pattern available ;-(. Also, in another PR I may update the ZookeeperInfoHandler. It returns JSON, now raw format, so why not use JSON directly? Plus it has some weridness around "clusterstate.json". # Tests Just added a BASIC test, didn't want to validate all the JSON formatting, but just enoguh to verify things wired together and hopefully catch any furthur regressions. -- 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]
