Hi Sage, It looks like common/PluginRegistry.{h,cc} are missing from 18ad8df1094db52c839dc6b2dc689fc882230acb
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:s...@newdream.net] >> 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 majord...@vger.kernel.org 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 majord...@vger.kernel.org >> 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 majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Loïc Dachary, Artisan Logiciel Libre
signature.asc
Description: OpenPGP digital signature