changes for adding properties to callog to track rac timestamp value #391
Conversation
|
This change adds an extra 80 to 95 bytes for every RAC_ID cal event. This event triggers only in case of any RAC maintenance. Sample: |
lib/workerpool.go
Outdated
| @@ -718,7 +718,7 @@ func (pool *WorkerPool) RacMaint(racReq racAct) { | |||
| if logger.GetLogger().V(logger.Verbose) { | |||
| logger.GetLogger().Log(logger.Verbose, "Rac maint activating, worker", i, pool.workers[i].pid, "exittime=", pool.workers[i].exitTime, now, window, pool.currentSize) | |||
| } | |||
| cnt++ | |||
| racMaintWorkers = append(racMaintWorkers, []interface{}{pool.moduleName, pool.ShardID, i, pool.workers[i].exitTime, pool.workers[i].exitTime - now}) | |||
There was a problem hiding this comment.
Change looks good to me. Instead of this, can we have an event inside this loop?
There was a problem hiding this comment.
Incorporated this review comment
No description provided.