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

Robert Stupp commented on CASSANDRA-10031:
------------------------------------------

Note: some profilers (at least JProfiler) get "annoyed" if thread names change 
as they intercept that change (calls to {{Thread.setName()}} to track threads. 
So I'd be careful with that change - especially when it comes to very frequent 
changes. It also produces additional heap pressure. (TBC: I won't fight against 
a change just because some profiles get annoyed - that's a problem of the 
profiler, not ours).

But giving threads more meaningful names is a good thing in general - just 
careful with too frequent changes. So, go ahead ;)

As an idea (no guarantees that it can work), {{Tracing}}/{{TraceSession}} 
already has (or could get) meaningful information that could help with 
debugging. But it might be too expensive - and, again, it's just a (possibly 
bad) idea.

> Name threads for improved ops/debugging
> ---------------------------------------
>
>                 Key: CASSANDRA-10031
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10031
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: T Jake Luciani
>            Priority: Minor
>              Labels: lhf
>             Fix For: 3.x
>
>
> We currently provide basic names for threads in threads like {{STREAM-IN-1}}  
> which gives some basic information about what the job of the thread is.  
> When looking at a log statement or jstack it's helpful to have this context.
> For our work stealing thread pool we share threads across all thread pools so 
> we lose this insight.  
> I'd like to propose we start using the Thread.currentThread().setName("")
> In different aspects of the code to improve insight as to what cassandra is 
> doing at any given moment.
>    * At a minimum in the start of each run() method.
>   Ideally for much finer grain things.
>    * In compaction include the partition name currently being working on.  
>    * In SP include the client ip
> Etc...



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

Reply via email to