On Fri, 25 Sep 2015, Mykola Golub wrote: > On Fri, Sep 25, 2015 at 07:09:53AM -0700, Sage Weil wrote: > > Hi Mykola, > > > > On Fri, 25 Sep 2015, Mykola Golub wrote: > > > What do you think about this implementation, which adds a dictionary > > > for pool options to pg_pool_t? > > > > > > https://github.com/ceph/ceph/pull/6081 > > > > > > Although #5922 has already been merged to master, I think it is still > > > not late to change scrub intervals to be stored in options? > > > > Yeah, I agree that something along these lines is better. It's too late > > to add this to infernalis, though.. I think we should revert the scrub > > interval options and then use the dictionary (post-infernalis). > > > > How does that sound? > > I sounds good to me. I will rebase #6081 against master after the > previous patch is reverted and mark it [DNM].
It's just this last commit, right? https://github.com/ceph/ceph/pull/6084 sage -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
