diff mbox

[v1,1/1] qapi/block-core: add doc describing GlusterServer vs. SocketAddress

Message ID 1471514967-15160-1-git-send-email-prasanna.kalever@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Prasanna Kumar Kalever Aug. 18, 2016, 10:09 a.m. UTC
Added documentation describing relation between GlusterServer and
SocketAddress qapi schemas.

Thanks to Markus Armbruster <armbru@redhat.com>

Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
---
 qapi/block-core.json | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Markus Armbruster Aug. 18, 2016, 12:32 p.m. UTC | #1
Prasanna Kumar Kalever <prasanna.kalever@redhat.com> writes:

> Added documentation describing relation between GlusterServer and
> SocketAddress qapi schemas.
>
> Thanks to Markus Armbruster <armbru@redhat.com>
>
> Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
> ---
>  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',
diff mbox

Patch

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
+#
+# 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.
+#
+# 2. GlusterServer lacks case 'fd', since gluster doesn't let you
+#    pass in a file descriptor.
+#
 # Since: 2.7
 ##
 { 'union': 'GlusterServer',