Revert back from oversight about rawDataRepacker and rawDataCollector for the BeamPixel DQM application

[dinardo]

In making a change to the code I, wrongly, realized that I could merge two sections of the code.
I was wrong because I confused rawDataCollector with rawDataRepacker.
I reverted back to the original form.
Sorry about that.

• Mauro.

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-24803/6749

[cmsbuild]

A new Pull Request was created by @dinardo (Mauro) for master.

It involves the following packages:

DQM/Integration

@kmaeshima, @cmsbuild, @andrius-k, @jfernan2, @schneiml can you please review it and eventually sign? Thanks.
@makortel, @threus, @batinkov, @thomreis 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

[cmsbuild]

The code-checks are being triggered in jenkins.

[dinardo]

This is a backport from PR #24802

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-24803/6750

[cmsbuild]

Pull request #24803 was updated. @kmaeshima, @cmsbuild, @andrius-k, @jfernan2, @schneiml can you please check and sign again.

[andrius-k]

please test

[andrius-k]

Hi @dinardo, could you please make sure that there is a name of the subsystem in the title of the PR?

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/30891/console Started: 2018/10/04 18:18

[cmsbuild]

+1
Tested at: 2731e80
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-24803/30891/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
2731e80
2ab6fd3
817710c
1143bd4
920d65e
22cd9c6
31560cf
a1b82c4
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-24803/30891/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-24803/30891/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 32
• DQMHistoTests: Total histograms compared: 3162909
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3162711
• DQMHistoTests: Total skipped: 197
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 8426.92 KiB( 31 files compared)
• DQMHistoSizes: changed ( 150.0,... ): 3370.768 KiB JetMET/HIJetValidation
• DQMHistoSizes: changed ( 158.0 ): 1685.384 KiB JetMET/HIJetValidation
• Checked 134 log files, 14 edm output root files, 32 DQM output files

[andrius-k]

Hi @dinardo, could you please make sure that there is a name of the subsystem in the title of the PR?

[dinardo]

Hi Anrdius,
ok it's done.

• Mauro.

[andrius-k]

+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)

[andrius-k]

@dinardo Thanks!

[fabiocos]

@dinardo this PR needs a 10_3_X backport I would say

[fabiocos]

+1