Re: widening Messenger::create

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I think it may better that Messenger impl can introspect itself by
"type", "policy", config and related infos?

On Fri, Jan 30, 2015 at 12:44 AM, Matt W. Benjamin <matt@xxxxxxxxxxxx> 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.
>
>>
>> sage
>
>
> --
> Matt Benjamin
> CohortFS, LLC.
> 315 West Huron Street, Suite 140A
> Ann Arbor, Michigan 48103
>
> http://cohortfs.com
>
> tel.  734-761-4689
> fax.  734-769-8938
> cel.  734-216-5309
> --
> 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



-- 
Best Regards,

Wheat
--
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




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux