Hi All, I created a separate PR for common plugin interface, corresponding to 2 Sage's commits and one my bug fix commit: https://github.com/ceph/ceph/pull/6696 Also I updated compression plugins commit with this patch: https://github.com/ceph/ceph/pull/6361 Igor will wait to update his PR (https://github.com/ceph/ceph/pull/6524) before we decide, how common plugin mechanism will be merged, to avoid multiple refactoring. So, please, take a look to my work, when you have some time. Also I would like to propose erasure code refactoring via new plugin mechanism. Please, let me know, if it's possible, if nobody work on it and I can do it. Thanks for reply. ------------------------------- Best regards, Alyona Kiseleva On Fri, Nov 13, 2015 at 4:18 PM, Sage Weil <sage@xxxxxxxxxxxx> wrote: > On Fri, 13 Nov 2015, Alyona Kiselyova wrote: >> Hi, >> I was working on pluggable compression interface in this work >> (https://github.com/ceph/ceph/pull/6361). In Igor's pull request was >> suggested to reuse common plugin infrastructure from unmerged >> wip-plugin branch. Now I'm working on adaptation of it, and as I see, >> I need only this two commits from it >> (https://github.com/ceph/ceph/commit/294bef3d12ec04d9febf1f850184be7653a4322cand >> https://github.com/ceph/ceph/commit/18ad8df1094db52c839dc6b2dc689fc882230acb). >> Sage, is it possible to make standalone pull request with them, or I >> must just to cherry pick them in my branch? > > Let's do a separate PR. > > Thanks! > sage > > >> Thanks for answer. >> ------------------------------- >> Best regards, >> Alyona Kiseleva >> >> >> On Tue, Nov 10, 2015 at 6:46 PM, Igor Fedotov <ifedotov@xxxxxxxxxxxx> wrote: >> > Hi All, >> > >> > a while ago we had some conversations here about adding compression support >> > for EC pools. >> > Here is corresponding pull request implementing this feature: >> > >> > https://github.com/ceph/ceph/pull/6524/commits >> > >> > Appropriate blueprint is at: >> > http://tracker.ceph.com/projects/ceph/wiki/Rados_-_at-rest_compression >> > >> > All comments and reviews are highly appreciated. >> > >> > Thanks, >> > Igor. >> > >> > -- >> > 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 >> -- >> 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 >> >> -- 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
