Centrality plugins and DQM centrality bugfixes

[yetkinyilmaz]

Bug fixes for :
#1564 (comment)
and
#2547 (comment)

[cmsbuild]

A new Pull Request was created by @yetkinyilmaz for CMSSW_5_3_X.

Centrality plugins and DQM centrality bugfixes

It involves the following packages:

DQM/Physics
RecoHI/HiCentralityAlgos

@thspeer, @danduggan, @rovere, @cmsbuild, @anton-a, @nclopezo, @deguio, @slava77, @Degano, @ojeda can you please review it and eventually sign? Thanks.
@azsigmon, @jazzitup, @yenjie, @kurtejung, @mandrenguyen, @richard-cms this is something you requested to watch as well.
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.
@smuzaffar you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[smuzaffar]

and what about RecoHI/HiCentralityAlgos/interface/CentralityTableHandler.h RecoHI/HiCentralityAlgos/src/CentralityTableHandler.cc
No one uses RecoHI/HiCentralityAlgos/interface/CentralityTableHandler.h except src/CentralityTableHandler.cc and it declares CentralityPopConProducer pluguin. So I would suggest to move both these 2 files to RecoHI/HiCentralityAlgos/plugins directory too

[smuzaffar]

Oops I see you have move RecoHI/HiCentralityAlgos/src/CentralityTableHandler.cc, but can you move RecoHI/HiCentralityAlgos/interface/CentralityTableHandler.h too? no one outside this package uses it.

[smuzaffar]

+tested
I will include it once RecoHI/HiCentralityAlgos/interface/CentralityTableHandler.h is also move to plugins directory

[smuzaffar]

@yetkinyilmaz, I am still waiting for this interface/CentralityTableHandler.h -> plugins/CentralityTableHandler.h move
@davidlange6, once we have this included then I think we are ready to build heavy ion re-reco release

[cmsbuild]

Pull request #2658 was updated. @thspeer, @danduggan, @rovere, @cmsbuild, @anton-a, @nclopezo, @deguio, @slava77, @Degano, @ojeda can you please check and sign again.

[cmsbuild]

Pull request #2658 was updated. @thspeer, @danduggan, @rovere, @cmsbuild, @anton-a, @nclopezo, @deguio, @slava77, @Degano, @ojeda can you please check and sign again.

[smuzaffar]

+tested
I am by passing this as it is just a technical change to fix the duplicate plugin issue+simple fix for #2547