[SOLR-7262] Registering / changing request handlers not thread safe

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.1
• Assignee: Yonik Seeley
• Reporter: Yonik Seeley
• Created: 18/Mar/15 01:08
• Updated: 15/Apr/15 00:30
• Resolved: 19/Mar/15 16:14

Description

Here's part of the javadoc for RequestHandlers.register()

   * This call is thread safe.
   * 
   * @return the previous handler at the given path or null
   */
  public SolrRequestHandler register( String handlerName, SolrRequestHandler handler ) {

But looking at the changes from SOLR-7073, this is clearly not the case (it's no longer thread safe). Was this thread safety removed on purpose or was it accidental?

Issue Links

• is broken by: SOLR-7073 Add an API to add a jar to a collection's classpath