Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

move modules to be thread_unsafe::DQMEDAnalyzers since they use non safe... #3554

Merged
merged 1 commit into from Apr 29, 2014
Merged

move modules to be thread_unsafe::DQMEDAnalyzers since they use non safe... #3554

merged 1 commit into from Apr 29, 2014

Conversation

deguio
Copy link
Contributor

@deguio deguio commented Apr 28, 2014

... ESProducts

@deguio
Copy link
Contributor Author

deguio commented Apr 28, 2014

+1
as soon as the tests confirm that the code runs fine.
@Dr15Jones

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes or unless it breaks tests. @nclopezo, @ktf can you please take care of it?

@Dr15Jones
Copy link
Contributor

Thanks!

@cmsbuild
Copy link
Contributor

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3554/1144/summary.html

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?

ktf added a commit that referenced this pull request Apr 29, 2014
@ktf
Merge pull request #3554 from deguio/makeModulesThreadUnsafe
82a7a47 
DQM -- move modules to be thread_unsafe::DQMEDAnalyzers since they use non safe...
@ktf ktf merged commit 82a7a47 into cms-sw:CMSSW_7_1_X Apr 29, 2014
@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3554/1144/summary.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
comparison-pending dqm-approved fully-signed tests-approved
Projects
None yet
Milestone
CMSSW_7_1_0_pre7
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@deguio @cmsbuild @Dr15Jones @ktf