From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Adam C. Emerson" Subject: Re: loadable objectstore Date: Mon, 22 Jun 2015 12:08:58 -0400 Message-ID: <20150622160858.GA13258@linuxbox.com> References: <1615100301.29.1434984131967.JavaMail.root@thunderbeast.private.linuxbox.com> <46879752.37.1434985621131.JavaMail.root@thunderbeast.private.linuxbox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from aa.linuxbox.com ([69.128.83.226]:1456 "EHLO aa.linuxbox.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751552AbbFVQPz (ORCPT ); Mon, 22 Jun 2015 12:15:55 -0400 Content-Disposition: inline In-Reply-To: Sender: ceph-devel-owner@vger.kernel.org List-ID: To: Varada Kari Cc: "Matt W. Benjamin" , Loic Dachary , ceph-devel , Sage Weil On 22/06/2015, Varada Kari wrote: > Hi Matt, > > Majority of the changes are segregating the files to corresponding shared object and creating a factory object. And the naming is mostly taken from Erasure-coding plugins. Want a good naming convention :-), hence a preliminary review. Do agree, we have lot of loadable interfaces, and I think we are in the way of making them on-demand (if possible) loadable modules. Since we're planning on having loadable modules become the new normal, I think it might be better to take 'Plugin' off the erasure coding classes than to add it to all the others. -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in