[lttng-dev] [RFC lttng-tools] Snapshot

[Jérémie Galarneau]

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 <dgoulet at efficios.com> 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
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
>



--
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com