[AFS3-std] review comments for draft-deason-afs3-getsizev2-03
Michael Meffie
mmeffie@sinenomine.net
Thu, 3 Apr 2014 14:54:32 -0400
On Thu, 3 Apr 2014 13:56:51 -0400
Benjamin Kaduk <kaduk@MIT.EDU> wrote:
> Hi all,
>
> I've read this document and have delightfully little to say about it.
> It's a pretty straightforward enhancement filling a long-standing void,
> which is good.
>
> The only real comment I might offer is that this is in last call jointly
> with a document defining 64-bit time types. Seciton 5.1 does mention
> future work for improving date representations (now concurrent work), but
> it doesn't seem that there is compelling reason to reopen this document to
> use the 64-bit time representation.
>
> It's unclear whether we want to suggest that the values of the GetSizeV2
> flags match the values of the DumpV2 flags, since they need not have the
> same flags. Given that there's only one flag at present and adding more
> would require some though, I don't think it matters either way.
>
> It also struck me in parts of Section 8 that we sometimes request things
> of the registry and at other times suggest actions for them. I don't know
> how important the distinction is; we could probably just request things
> always.
>
> Anyway, this document is fine and I would be happy if it was published
> as-is. I do not think any changes are necessary.
Thank you Ben, for the review and the comments.
Thanks,
Mike
--
Michael Meffie <mmeffie@sinenomine.net>