Skip to content

Commit

Permalink
[controller] add cluster name to the 'Venice-Admin-Execution-Task' th… (
Browse files Browse the repository at this point in the history
#684)

[controller] add cluster name to the 'Venice-Admin-Execution-Task' threads

In controller, each cluster has one corresponding AdminConsumptionTask and
multiple 'Venice-Admin-Execution-Task' to consume the admin messages and
then execute the admin operations. Today, it is not easy to differentiate
which cluster one execution task belongs to, if one analyze controller logs
or Java heap dump. This change appends the cluster name to the existing
thread name prefix to make it distinguishable.
  • Loading branch information
lluwm authored Oct 17, 2023
1 parent 486cb94 commit 270349c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@ public AdminConsumptionTask(
60,
TimeUnit.SECONDS,
new LinkedBlockingQueue<>(MAX_WORKER_QUEUE_SIZE),
new DaemonThreadFactory("Venice-Admin-Execution-Task"));
new DaemonThreadFactory(String.format("Venice-Admin-Execution-Task-%s", clusterName)));
this.undelegatedRecords = new LinkedList<>();
this.stats.setAdminConsumptionFailedOffset(failingOffset);
this.pubSubTopicRepository = pubSubTopicRepository;
Expand Down

0 comments on commit 270349c

Please sign in to comment.