[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 1/1] qapi/block-core: add doc describing Glus
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v1 1/1] qapi/block-core: add doc describing GlusterServer vs. SocketAddress |
Date: |
Thu, 18 Aug 2016 14:32:02 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Prasanna Kumar Kalever <address@hidden> writes:
> Added documentation describing relation between GlusterServer and
> SocketAddress qapi schemas.
>
> Thanks to Markus Armbruster <address@hidden>
>
> Signed-off-by: Prasanna Kumar Kalever <address@hidden>
> ---
> qapi/block-core.json | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/qapi/block-core.json b/qapi/block-core.json
> index 5e2d7d7..5305562 100644
> --- a/qapi/block-core.json
> +++ b/qapi/block-core.json
> @@ -2121,6 +2121,15 @@
> #
> # @tcp: host address and port number
> #
> +# GlusterServer vs. SocketAddress
Suggest:
# This is similar to SocketAddress, only simpler:
> +#
> +# 1. GlusterServer is a flat union, SocketAddress is a simple union.
> +# A flat union is nicer over simple for the reason it avoids
> +# nesting (i.e. more {}) on the wire.
Perhaps "nicer than simple because it"
> +#
> +# 2. GlusterServer lacks case 'fd', since gluster doesn't let you
> +# pass in a file descriptor.
> +#
Suggest:
# GlusterServer is actually not Gluster-specific, and could be
# compatibily evolved into a replacement for SocketAddress.
> # Since: 2.7
> ##
> { 'union': 'GlusterServer',