code-cleaning in RecoMET/METProducers

[TaiSakuma]

The constructor and the method produce() execute nearly entirely different sets of lines of code based on METtype.
https://github.com/TaiSakuma/cmssw/blob/CMSSW_7_0_0_pre9/RecoMET/METProducers/src/METProducer.cc

Instead of having a single producer which executes different lines based on METtype, we should have multiple producers, one producer for each METtype.

I also changed cfi files to use new producers.

I didn't delete METProducer yet. I plan to make another request in the future that will delete METProducer.

[cmsbuild]

A new Pull Request was created by @TaiSakuma (Tai Sakuma) for CMSSW_7_0_X.

code-cleaning in RecoMET/METProducers

It involves the following packages:

RecoMET/METProducers

@nclopezo, @cmsbuild, @thspeer, @slava77 can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@ktf you are the release manager for this.

[cmsbuild]

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

[cmsbuild]

Pull request #1745 was updated. @nclopezo, @cmsbuild, @thspeer, @slava77 can you please check and sign again.

[cmsbuild]

Pull request #1745 was updated. @nclopezo, @cmsbuild, @thspeer, @slava77 can you please check and sign again.

[cmsbuild]

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

[thspeer]

+1
tested d9e38aa in CMSSW_7_0_X_2013-12-11-0200_1745
No difference in reco as expected, based on RelMon and reco script
Would be good indeed to remove METProducer so as not to create confusion.

[cmsbuild]

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

[cmsbuild]

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

[slava77]

I'm curious why is there another run by jenkins?
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1745/1702/summary.html
and now
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1745/1734/summary.html