Published a pull request for extracting Group.cc https://github.com/ceph/ceph/pull/11070 Please review. On Tue, Sep 13, 2016 at 5:33 AM, Jason Dillaman <jdillama@xxxxxxxxxx> wrote: > On Mon, Sep 12, 2016 at 7:09 PM, Victor Denisov <vdenisov@xxxxxxxxxxxx> wrote: >> I also think, probably, moving existing group api to Group.cc can be >> done in a separate pull request. > > Agreed -- no worries. > >> On Mon, Sep 12, 2016 at 4:06 PM, Victor Denisov <vdenisov@xxxxxxxxxxxx> wrote: >>> Another quick question. >>> Do you think it makes sense to introduce snapshot namespaces in a pull >>> request and review it first? >>> It looks like a self sufficient change that we can merge before >>> introducing snapshots. > > Yes, I think that would be a good idea to simplify the PR and get it > merged potentially quicker. > > -- > Jason -- 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
