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

Ramkumar Aiyengar commented on SOLR-7270:
-----------------------------------------

Can't our exceptions include this information? Just that modifying thread names 
seems unintuitive to someone new looking at the code. The name kinda identifies 
the thread to the developer. Agreed this should be a lesser when its an 
executor and its just an implementation detail that threads are reused.. But if 
there's no better way, let's go for this (In such a case, I am more inclined to 
have the name be just set instead of us parsing using | etc., that logic can be 
done by the DynamicRunnable, so it sets updateProcessor-host1 or something like 
that)

> Add URL of remote host in distrib update thread names to improve 
> debuggability via stack traces.
> ------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-7270
>                 URL: https://issues.apache.org/jira/browse/SOLR-7270
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Mark Miller
>            Assignee: Mark Miller
>         Attachments: SOLR-7270.patch, SOLR-7270.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to