Reduced ES alcareco size

[crovelli]

This PR is to remove two unused collections from the output of the EcalESAlign Alcareco.
Once integrated in 10.2.x, we would like to back-port it to 10.1.x as well, as per A.Kumar's request

[cmsbuild]

@crovelli, CMSSW_10_2_X branch is closed for direct updates. cms-bot is going to move this PR to master branch.
In future, please use cmssw master branch to submit your changes.

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23265/4756

[cmsbuild]

A new Pull Request was created by @crovelli for master.

It involves the following packages:

Calibration/EcalAlCaRecoProducers

@cmsbuild, @franzoni, @arunhep, @cerminar, @lpernie can you please review it and eventually sign? Thanks.
@mmusich, @tocheng, @argiro this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[lpernie]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/28093/console Started: 2018/05/21 23:04

[cmsbuild]

+1
Tested at: d8dbe7d
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23265/28093/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 31
• DQMHistoTests: Total histograms compared: 2901712
• DQMHistoTests: Total failures: 2
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2901520
• DQMHistoTests: Total skipped: 190
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 30 files compared)
• Checked 128 log files, 14 edm output root files, 31 DQM output files

[arunhep]

@crovelli thanks for this PR. What is the impact on the size/event because of this ?

[arunhep]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[arunhep]

@crovelli can we also please have the backport of this PR in 10.1.X?
we should aim to have it in next 10.1.X release which should be made in coming days for special beta=90m run.

[crovelli]

@arunhep, I did not check myself. According to our preshower experts this will save 4%. There are other actions which may help in reducing the event size but they require more studies and time.

…

On Tue, May 22, 2018 at 2:26 PM, Arun Kumar ***@***.***> wrote: @crovelli <https://github.com/crovelli> thanks for this PR. What is the impact on the size/event because of this ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#23265 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEhG3YkHIM2JbgBlSnyrapQcyJg-k8PDks5t1AP7gaJpZM4UG-MD> .

[crovelli]

done: #23278

…

On Tue, May 22, 2018 at 2:28 PM, Arun Kumar ***@***.***> wrote: @crovelli <https://github.com/crovelli> can we also please have the backport of this PR in 10.1.X? we should aim to have it in next 10.1.X release which should be made in coming days for special beta=90m run. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#23265 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEhG3eCSTDkqQ66bO1Z2C1pu4N1IOtMFks5t1ARdgaJpZM4UG-MD> .

[fabiocos]

+1