On Sat, 11 Apr 2015, Loic Dachary wrote:
> Hi Sage,
>
> It looks like common/PluginRegistry.{h,cc} are missing from
> 18ad8df1094db52c839dc6b2dc689fc882230acb
whoops pushed! :)
sage
>
> Cheers
>
> On 11/04/2015 16:51, Sage Weil wrote:
> > On Sat, 11 Apr 2015, Varada Kari wrote:
> >> Hi Matt/ Sage,
> >>
> >> Thanks for sharing the pull request. Will work on the items what sage
> >> was mentioning in the comments along with the items in the tracker.
> >
> > I pushed a wip-plugin branch with the pattern I had in mind... does this
> > make sense? It's only compile tested, and still needs preload() methods
> > implemented so that long-running daemons (or perhaps others?) can preload
> > it all at once.
> >
> > Ideally we would replace ErasurePluginRegistry with this too, which means
> > the naming of the .so's needs to be a bit more flexible as the EC plugin
> > scheme is something like /usr/lib/ceph/erasure-code/libec_$name.so (not a
> > strict function of type and name). Or actually, we can just rename the
> > EC .so's when this goes in? Since they're explicitly incompatible
> > between versions there is no reason the names need to remain
> > consistent...
> >
> > sage
> >
> >>
> >> Varada
> >>
> >> -----Original Message-----
> >> From: Sage Weil [mailto:[email protected]]
> >> Sent: Saturday, April 11, 2015 4:16 AM
> >> To: Matt W. Benjamin
> >> Cc: Varada Kari; ceph-devel
> >> Subject: Re: loadable objectstore
> >>
> >> On Fri, 10 Apr 2015, Matt W. Benjamin wrote:
> >>> Hi Varada,
> >>>
> >>> I pushed branch hammer-osfactory to
> >>> https://github.com/linuxbox2/ceph.git
> >>>
> >>> It can at least provide a starting point for discussion, if not a
> >>> jumping off point; I do think it's basically what Sage described.
> >>
> >> I made a few comments on the pull request. I think a factory class is the
> >> way to go, but we should be able to make the plugin loading part generic
> >> and not interface specific at all...
> >>
> >> sage
> >>
> >>>
> >>> I was able to update the CMake build system to build the tree, I
> >>> haven't attempted Autotools. :(
> >>>
> >>> Regards,
> >>>
> >>> Matt
> >>>
> >>> --
> >>> 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 [email protected] More majordomo
> >>> info at http://vger.kernel.org/majordomo-info.html
> >>>
> >>>
> >>
> >> ________________________________
> >>
> >> PLEASE NOTE: The information contained in this electronic mail message is
> >> intended only for the use of the designated recipient(s) named above. If
> >> the reader of this message is not the intended recipient, you are hereby
> >> notified that you have received this message in error and that any review,
> >> dissemination, distribution, or copying of this message is strictly
> >> prohibited. If you have received this communication in error, please
> >> notify the sender by telephone or e-mail (as shown above) immediately and
> >> destroy any and all copies of this message in your possession (whether
> >> hard copies or electronically stored copies).
> >>
> >> --
> >> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> >> the body of a message to [email protected]
> >> More majordomo info at http://vger.kernel.org/majordomo-info.html
> >>
> >>
> > --
> > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> > the body of a message to [email protected]
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
> --
> Loïc Dachary, Artisan Logiciel Libre
>
>
