On Thu, 29 Jan 2015, Matt W. Benjamin wrote: > Hi, > > ----- "Sage Weil" <sage@xxxxxxxxxxxx> wrote: > > > On Thu, 29 Jan 2015, Matt W. Benjamin wrote: > > > Hi, > > > > > > I'm looking at ways to pass potentially messenger-specific options > > > through the Messenger factory. The immediate parameter I need to > > > provide is the number of Accelio portals to set up (this should vary > > > by Messenger instance), but I expect others will be needed in > > > future. I don't think this fits cleanly on to the Policy mechanism, > > > though I looked at that. I think I'd like to provide a generalized > > > argument to the Messenger factory, which can do the right thing for > > > a given Messenger (e.g., constructor). > > > > In the past we've mostly gotten away with impl-specific config > > options. > > Is that not sufficient here? Because you need to pass different > > parameters to different instances maybe? > > Yes, exactly. Hmm, in that case I'd say if you can make it as generic as possible that's about all we can do that this point. Just a single arg now? We can easily expand the interface to a param struct later. (Or do that now if you prefer.. whatever works and looks reasonable!) sage -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
