[ 
https://issues.apache.org/jira/browse/SOLR-13310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16788391#comment-16788391
 ] 

Mark Miller commented on SOLR-13310:
------------------------------------

I got rid of the ugly exceptions that allowed static executors like this to 
live because those exceptions hid all sorts of things (things like allowing 30 
seconds for threads to die and such).

No clue how a thread pool executor is treated special due to error handling.

Executors like this should live on the core container - I think I made a 
general use executor like that for the starburst branch, but here I just 
grabbed what is called the update executor but is actually more accurately the 
non http shard handler misc executor as currently used.


> facet.threads is using the updateExecutor
> -----------------------------------------
>
>                 Key: SOLR-13310
>                 URL: https://issues.apache.org/jira/browse/SOLR-13310
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Hoss Man
>            Priority: Major
>
> Had a WTF moment skimming some SimpleFacets code today...
> {code}
>     this.facetExecutor = 
> req.getCore().getCoreContainer().getUpdateShardHandler().getUpdateExecutor();
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to