qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Nbd] [PATCH] Further tidy-up on block status


From: Wouter Verhelst
Subject: Re: [Qemu-devel] [Nbd] [PATCH] Further tidy-up on block status
Date: Sat, 17 Dec 2016 09:34:36 +0100
User-agent: NeoMutt/20161126 (1.7.1)

On Fri, Dec 16, 2016 at 04:25:27PM +0000, Alex Bligh wrote:
> 
> > On 16 Dec 2016, at 15:52, Wouter Verhelst <address@hidden> wrote:
> > 
> > On Thu, Dec 15, 2016 at 05:34:48PM +0000, Alex Bligh wrote:
> >> 
> >>> On 15 Dec 2016, at 16:49, Wouter Verhelst <address@hidden> wrote:
> >>> 
> >>>> Because the namespaces and leaf-names are already restricted to
> >>>> non-whitespace characters. I thought having tabs, line feeds,
> >>>> returns, em-space, en-space etc. was not particularly useful.
> >>>> I could be persuaded to relent re spaces.
> >>> 
> >>> I could imagine that the context might include as part of its name a
> >>> user-defined bit. If we're going to disallow whitespace, then that would
> >>> mean namespaces would have to do all sorts of escaping etc. I don't
> >>> think that's a good idea.
> >> 
> >> So to be clear do you want to include all whitespace
> >> everywhere? Or just to the right of the colon in queries?
> > 
> > Just in queries. I agree that in namespace names, it doesn't make much
> > sense.
> 
> I've changed it so the returns from queries can now have spaces to
> the right of the colon if they aren't returning complete context names.
> Obviously if they are returning context names, by the definition of
> leaf-names they can't have spaces in the leaf-name.

Sorry, that still doesn't alleviate my concerns.

Why are we restricting what metadata contexts can use for their names?
We don't restrict what export names can use either, and I could imagine
a metadata context wanting to use an export name as a metadata context
name (e.g., "provide a bitmap on the diff between this export and the
export defined by name XYZ").

As soon as we get to the right of the colon, what sits there is defined
by the namespace, not by us. It doesn't feel right to add too many
restrictions.

I think the whole query string should be a "string" as laid out earlier
in the document. Here, it actually *does* make sense for a string to be
very long, because it's not meant to be human-readable.

I've therefore removed that restriction as well as the "255 bytes max"
one that you added, since I don't think they make much sense. That
doesn't mean I can't be convinced otherwise by good arguments, but
they'd have to be very good ones ;-)

> The queries themselves can consist of anything (currently),
> save that they must begin with a namespace and a colon.

Sure.

-- 
< ron> I mean, the main *practical* problem with C++, is there's like a dozen
       people in the world who think they really understand all of its rules,
       and pretty much all of them are just lying to themselves too.
 -- #debian-devel, OFTC, 2016-02-12



reply via email to

[Prev in Thread] Current Thread [Next in Thread]