Looks great! A few very minor points: In the "Motivation" section, English mistake: "could be particularly useful *with in* flight recorder mode" should be "could be particularly useful *in* flight recorder mode". Also, "informationS" is generally regarded as incorrect. Information has no plural form.
In the "Proposed Solution" section, "the lttng command line UI new command is presented" should be "the new lttng command-line UI command is presented". I'd propose rewriting the sentence "Unlike the other lttng command, this one uses the git alike UI but in the form of the object first and action after where previous command only use action such as "enable-event" as "This new command uses a git-alike UI, but in the form of the object first followed by the desired action, whereas other commands only use actions such as "enable-event"". I'm not sure I would use an hyphen in "lttng snapshot list-output" when an output alias is present. That's an entirely subjective but it would seem more consistent with git (git remote -v, for instance). Could you clarify what the expected behavior would be when no session name is provided to "lttng snapshot record"? I guess it would capture a snapshot for every session created with the --no-output option. Also, what happens if a single URL is provided and no session name is given (with multiple active flight recorder sessions)? Thanks, Jérémie On Thu, Apr 11, 2013 at 11:16 AM, David Goulet <[email protected]> wrote: > > Hi everyone, > > This is the initial RFC for snapshot in lttng-tools. Once accepted, it > will be available in "doc/proposals/0006-lttng-snapshot.txt". > > Any questions, comments or fixes are VERY welcome. There are probably > English mistakes so feel free to point it out to me. > > Thanks! > David > > _______________________________________________ > lttng-dev mailing list > [email protected] > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev > -- Jérémie Galarneau EfficiOS Inc. http://www.efficios.com _______________________________________________ lttng-dev mailing list [email protected] http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
