Hi Slava,

While this code will not be run in regular production application,
it will be used to prepare production databases.

In principle, I agree that there is probably a better place for the
DB read/write headers. However, it is surely not "test". Why don't
we keep it as it is for pre8 (since everything works, and the deadline
is fast approaching), and I will find a better place for it before pre9.

Regards,
Igor

On 05/13/2014 09:37 PM, Slava Krutelyov wrote:
>
> In RecoLocalCalo/HcalRecAlgos/interface/BoostIODBReader.h:
>
> > @@ -0,0 +1,80 @@
> > +#ifndef RecoLocalCalo_HcalRecAlgos_BoostIODBReader_h
>
> It's not going to be run in regular production application, only by experts once in a blue moon.
> This is pretty much the definition of what should be in the test.
> This comment refers to the instances defined in the plugin as well.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/cms-sw/cmssw/pull/3806/files#r12618804>.
>