[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] block/gluster: rename [server, volname, ima
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] block/gluster: rename [server, volname, image] -> [host, volume, path] |
Date: |
Thu, 12 Nov 2015 13:28:33 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 11/12/2015 03:22 AM, Prasanna Kumar Kalever wrote:
> this patch is very much be meaningful after next patch which adds multiple
> gluster servers support. After that,
>
> an example is, in 'servers' tuple values we use 'server' variable for key
Awkward line break mid-sentence. The commit message alone is not a
reason to hold up this patch, so maybe the maintainer can adjust it.
> 'host' in the code, it will be quite messy to have colliding names for
> variables, so to maintain better readability and makes it consistent with
> other
> existing code as well as the input keys/options, this patch renames the
> following variables
I'd suggest replacing everything up to here with:
A future patch will add support for multiple gluster servers. Our
existing terminology is a bit unusual in relation to what names are used
by other networked devices, and doesn't map very well to the terminology
we expect to use for multiple servers. Therefore, rename the following
options:
> 'server' -> 'host'
> 'image' -> 'path'
> 'volname' -> 'volume'
>
> Signed-off-by: Prasanna Kumar Kalever <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> ---
> block/gluster.c | 54 +++++++++++++++++++++++++++---------------------------
> 1 file changed, 27 insertions(+), 27 deletions(-)
R-b still stands on this one.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/4] block/gluster: add support for multiple gluster servers, Prasanna Kumar Kalever, 2015/11/12
- [Qemu-devel] [PATCH 1/4] block/gluster: rename [server, volname, image] -> [host, volume, path], Prasanna Kumar Kalever, 2015/11/12
- Re: [Qemu-devel] [PATCH 1/4] block/gluster: rename [server, volname, image] -> [host, volume, path],
Eric Blake <=
- [Qemu-devel] [PATCH 2/4] block/gluster: code cleanup, Prasanna Kumar Kalever, 2015/11/12
- [Qemu-devel] [PATCH 3/4] block/gluster: using new qapi schema, Prasanna Kumar Kalever, 2015/11/12
- Re: [Qemu-devel] [PATCH 3/4] block/gluster: using new qapi schema, Jeff Cody, 2015/11/12
- Re: [Qemu-devel] [PATCH 3/4] block/gluster: using new qapi schema, Eric Blake, 2015/11/12
- Re: [Qemu-devel] [PATCH 3/4] block/gluster: using new qapi schema, Eric Blake, 2015/11/12
- Re: [Qemu-devel] [PATCH 3/4] block/gluster: using new qapi schema, Eric Blake, 2015/11/12
- Re: [Qemu-devel] [PATCH 3/4] block/gluster: using new qapi schema, Markus Armbruster, 2015/11/13
- [Qemu-devel] [PATCH 4/4] block/gluster: add support for multiple gluster servers, Prasanna Kumar Kalever, 2015/11/12