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