Dude .... you've got to at least run "ant test" before you commit ...
TestWriterPerf can't even compile now.
This breaks anyone that was explicitly passing null to the existing
getRequestHandler(String) method (like TestWriterPerf) ... but besides
that it also seems really weird to have a method that takes in a class and
arbitrarily returns the first instance of that class found -- especially
since it's iterating over the values in a Map, so if someone has multiple
instances of a handler they could get a different one each time this
method is called.
making a method that returns *all* the handlers of that type seems like a
better general feature for SolrCore/RequestHandlers, and then in this
specific use case of the replication JSP can be as arbitrary as it wants.
i've got a fix i'll commit as soon as my tests finish...