Project

General

Profile

Feature #15658

Milestone #15372: art multi-threading phase 1

Feature #15374: Modifying all framework-provided services to be thread safe.

Make TimeTracker service thread safe

Added by Kyle Knoepfel over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
02/24/2017
Due date:
% Done:

100%

Estimated time:
8.00 h
Spent time:
Scope:
Internal
Experiment:
-
SSI Package:
art
Duration:

Description

Depends on cet::Ntuple, which is not yet thread safe. The TimeTracker service would need to be thread aware for calculating time differences, which are then inserted into the database. We should investigate if using CMSSW's StallMonitor, and suitably adjusting it to use SQLite, could be of benefit.


Related issues

Blocked by cetlib - Feature #15672: Make SQLite cet::Ntuple facility thread safeClosed2017-02-27

History

#1 Updated by Kyle Knoepfel over 2 years ago

  • Blocked by Feature #15672: Make SQLite cet::Ntuple facility thread safe added

#2 Updated by Kyle Knoepfel over 2 years ago

  • % Done changed from 0 to 40

Removed TimeTracker header so that users cannot explicitly interact with the TimeTracker service. All that remains is to make the TimeTracker callbacks thread-safe, which relies inherently on cet::Ntuple.

#3 Updated by Kyle Knoepfel over 2 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 40 to 100

Thread-safety achieved by representing time point data as an std::vector<PerScheduleData>, where the vector grows to the number of schedules specified by the user. Right now, there is a placeholder number of 1 schedule, which can be replaced by the necessary facility that communicates how many schedules have been configured by the user.

Implemented with commit art:f4d403e7.

#4 Updated by Kyle Knoepfel over 2 years ago

  • Target version set to 1209

#5 Updated by Kyle Knoepfel about 2 years ago

  • Target version changed from 1209 to 2.07.01

#6 Updated by Kyle Knoepfel about 2 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF