gresrun/jesque — pull request #17 by trimbo (2012-04-06 20:58:17+00:00)
Title: Thread renaming removed

Hi Greg. Thanks for Jesque, we've gotten a lot of awesome use from it.

I've been looking at some profiling information today and it turns out that thread renaming incurs an unreasonable mount of overhead when done at scale. We're seeing it done several hundred thousand to millions of times and this ends up taking 6-8% of total CPU usage for our work and an even larger percentage of the jesque worker code (I think 20-30%).

I'm hoping you'll take this commit that removes thread renaming. I understand it can be useful for monitoring and debugging, but given the performance implications I thought it best to remove it altogether. Your thoughts?

- Chris