Clojure

Set thread names on agent thread pools

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: Release 1.3
  • Component/s: None
  • Labels:
    None
  • Approval:
    Ok

Description

It���s a best practice to name the threads in an executor thread pool with a custom ThreadFactory so that the purpose of these threads is clear in thread dumps and other runtime operational tools. By default these threads are currently called something like "pool-%d-thread-%d", and this is what you���ll see for the agent send thread pools.

I created a patch to do this with thread names like:

  • clojure-agent-send-pool-%d ��� should be fixed # of threads
  • clojure-agent-send-off-pool-%d ��� will be added and removed over time

The patch is attached and I have a signed CA.

Activity

Hide
Assembla Importer added a comment -
Show
Assembla Importer added a comment - stu said: [file:cUhT1C1uCr35jCeJe5cbLA]
Hide
Assembla Importer added a comment -

stu said: Second patch simply recreates first patch in the correct form. Please follow the instructions at http://clojure.org/patches when creating patches.

Show
Assembla Importer added a comment - stu said: Second patch simply recreates first patch in the correct form. Please follow the instructions at http://clojure.org/patches when creating patches.
Hide
Assembla Importer added a comment -

stu said: Updating tickets (#276, #280, #378, #437, #448)

Show
Assembla Importer added a comment - stu said: Updating tickets (#276, #280, #378, #437, #448)

People

  • Assignee:
    Unassigned
    Reporter:
    Anonymous
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: