From: Mykola Golub <to.my.trociny@gmail.com>
To: Sage Weil <sweil@redhat.com>
Cc: Mykola Golub <mgolub@mirantis.com>,
GuangYang <yguang11@outlook.com>,
"sjust@redhat.com" <sjust@redhat.com>,
"ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>,
David Zafman <dzafman@redhat.com>
Subject: Re: Pool setting for recovery priority
Date: Fri, 25 Sep 2015 20:50:43 +0300 [thread overview]
Message-ID: <20150925175042.GA3831@gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1509250708531.13265@cobra.newdream.net>
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].
--
Mykola Golub
next prev parent reply other threads:[~2015-09-25 17:50 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-16 15:48 Pool setting for recovery priority GuangYang
2015-09-16 16:23 ` Sage Weil
2015-09-16 17:58 ` GuangYang
2015-09-21 13:32 ` Mykola Golub
2015-09-25 11:44 ` Mykola Golub
2015-09-25 14:09 ` Sage Weil
2015-09-25 17:50 ` Mykola Golub [this message]
2015-09-25 18:02 ` Sage Weil
2015-09-25 18:09 ` Mykola Golub
2015-09-29 17:23 ` GuangYang
2015-09-29 18:47 ` Mykola Golub
2015-09-29 20:57 ` GuangYang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150925175042.GA3831@gmail.com \
--to=to.my.trociny@gmail.com \
--cc=ceph-devel@vger.kernel.org \
--cc=dzafman@redhat.com \
--cc=mgolub@mirantis.com \
--cc=sjust@redhat.com \
--cc=sweil@redhat.com \
--cc=yguang11@outlook.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.