* [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command @ 2019-08-27 16:56 Garima Singh via GitGitGadget 2019-08-27 16:56 ` [PATCH 1/1] " Garima Singh via GitGitGadget 2019-09-10 13:54 ` [PATCH 0/1] " Garima Singh 0 siblings, 2 replies; 5+ messages in thread From: Garima Singh via GitGitGadget @ 2019-08-27 16:56 UTC (permalink / raw) To: git; +Cc: Junio C Hamano Emit trace2_cmd_mode() messages for each commit-graph sub-command. The commit graph commands were in flux when trace2 was making it's way to git. Now that we have enough sub-commands in commit-graph, we can label the various modes within them. Distinguishing between read, write and verify is a great start. Signed-off-by: Garima Singh garima.singh@microsoft.com [garima.singh@microsoft.com] CC: jeffhost@microsoft.com, stolee@gmail.com, garimasigit@gmail.com, avarab@gmail.com Garima Singh (1): commit-graph: emit trace2 cmd_mode for each sub-command builtin/commit-graph.c | 6 ++++++ 1 file changed, 6 insertions(+) base-commit: 745f6812895b31c02b29bdfe4ae8e5498f776c26 Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-323%2Fgarimasi514%2FcoreGit-commit-graph-trace2-v1 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-323/garimasi514/coreGit-commit-graph-trace2-v1 Pull-Request: https://github.com/gitgitgadget/git/pull/323 -- gitgitgadget ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] commit-graph: emit trace2 cmd_mode for each sub-command 2019-08-27 16:56 [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command Garima Singh via GitGitGadget @ 2019-08-27 16:56 ` Garima Singh via GitGitGadget 2019-09-10 13:54 ` [PATCH 0/1] " Garima Singh 1 sibling, 0 replies; 5+ messages in thread From: Garima Singh via GitGitGadget @ 2019-08-27 16:56 UTC (permalink / raw) To: git; +Cc: Junio C Hamano, Garima Singh From: Garima Singh <garima.singh@microsoft.com> Emit trace2_cmd_mode() messages for each commit-graph sub-command. The commit graph commands were in flux when trace2 was making it's way to git. Now that we have enough sub-commands in commit-graph, we can label the various modes within them. Distinguishing between read, write and verify is a great start. Signed-off-by: Garima Singh <garima.singh@microsoft.com> --- builtin/commit-graph.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/builtin/commit-graph.c b/builtin/commit-graph.c index 57863619b7..ef68b26744 100644 --- a/builtin/commit-graph.c +++ b/builtin/commit-graph.c @@ -58,6 +58,8 @@ static int graph_verify(int argc, const char **argv) OPT_END(), }; + trace2_cmd_mode("verify"); + argc = parse_options(argc, argv, NULL, builtin_commit_graph_verify_options, builtin_commit_graph_verify_usage, 0); @@ -102,6 +104,8 @@ static int graph_read(int argc, const char **argv) OPT_END(), }; + trace2_cmd_mode("read"); + argc = parse_options(argc, argv, NULL, builtin_commit_graph_read_options, builtin_commit_graph_read_usage, 0); @@ -183,6 +187,8 @@ static int graph_write(int argc, const char **argv) split_opts.max_commits = 0; split_opts.expire_time = 0; + trace2_cmd_mode("write"); + argc = parse_options(argc, argv, NULL, builtin_commit_graph_write_options, builtin_commit_graph_write_usage, 0); -- gitgitgadget ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command 2019-08-27 16:56 [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command Garima Singh via GitGitGadget 2019-08-27 16:56 ` [PATCH 1/1] " Garima Singh via GitGitGadget @ 2019-09-10 13:54 ` Garima Singh 2019-09-11 18:17 ` Jeff Hostetler 1 sibling, 1 reply; 5+ messages in thread From: Garima Singh @ 2019-09-10 13:54 UTC (permalink / raw) To: Garima Singh via GitGitGadget, git; +Cc: Junio C Hamano Ping :) Any thoughts on this? On 8/27/2019 12:56 PM, Garima Singh via GitGitGadget wrote: > Emit trace2_cmd_mode() messages for each commit-graph sub-command. > > The commit graph commands were in flux when trace2 was making it's way to > git. Now that we have enough sub-commands in commit-graph, we can label the > various modes within them. Distinguishing between read, write and verify is > a great start. > > Signed-off-by: Garima Singh garima.singh@microsoft.com > [garima.singh@microsoft.com] > > CC: jeffhost@microsoft.com, stolee@gmail.com, garimasigit@gmail.com, > avarab@gmail.com > > Garima Singh (1): > commit-graph: emit trace2 cmd_mode for each sub-command > > builtin/commit-graph.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > > base-commit: 745f6812895b31c02b29bdfe4ae8e5498f776c26 > Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-323%2Fgarimasi514%2FcoreGit-commit-graph-trace2-v1 > Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-323/garimasi514/coreGit-commit-graph-trace2-v1 > Pull-Request: https://github.com/gitgitgadget/git/pull/323 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command 2019-09-10 13:54 ` [PATCH 0/1] " Garima Singh @ 2019-09-11 18:17 ` Jeff Hostetler 2019-10-02 8:02 ` Johannes Schindelin 0 siblings, 1 reply; 5+ messages in thread From: Jeff Hostetler @ 2019-09-11 18:17 UTC (permalink / raw) To: Garima Singh, Garima Singh via GitGitGadget, git; +Cc: Junio C Hamano On 9/10/2019 9:54 AM, Garima Singh wrote: > Ping :) Any thoughts on this? > > On 8/27/2019 12:56 PM, Garima Singh via GitGitGadget wrote: >> Emit trace2_cmd_mode() messages for each commit-graph sub-command. >> >> The commit graph commands were in flux when trace2 was making it's way to >> git. Now that we have enough sub-commands in commit-graph, we can >> label the >> various modes within them. Distinguishing between read, write and >> verify is >> a great start. >> >> Signed-off-by: Garima Singh garima.singh@microsoft.com >> [garima.singh@microsoft.com] >> >> CC: jeffhost@microsoft.com, stolee@gmail.com, garimasigit@gmail.com, >> avarab@gmail.com >> >> Garima Singh (1): >> commit-graph: emit trace2 cmd_mode for each sub-command >> >> builtin/commit-graph.c | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> >> base-commit: 745f6812895b31c02b29bdfe4ae8e5498f776c26 >> Published-As: >> https://github.com/gitgitgadget/git/releases/tag/pr-323%2Fgarimasi514%2FcoreGit-commit-graph-trace2-v1 >> >> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git >> pr-323/garimasi514/coreGit-commit-graph-trace2-v1 >> Pull-Request: https://github.com/gitgitgadget/git/pull/323 >> Looks good to me. Thanks! Signed-off-by: Jeff Hostetler <jeffhost@microsoft.com> ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command 2019-09-11 18:17 ` Jeff Hostetler @ 2019-10-02 8:02 ` Johannes Schindelin 0 siblings, 0 replies; 5+ messages in thread From: Johannes Schindelin @ 2019-10-02 8:02 UTC (permalink / raw) To: Jeff Hostetler Cc: Garima Singh, Garima Singh via GitGitGadget, git, Junio C Hamano [-- Attachment #1: Type: text/plain, Size: 1574 bytes --] Hi, On Wed, 11 Sep 2019, Jeff Hostetler wrote: > On 9/10/2019 9:54 AM, Garima Singh wrote: > > Ping :) Any thoughts on this? > > > > On 8/27/2019 12:56 PM, Garima Singh via GitGitGadget wrote: > > > Emit trace2_cmd_mode() messages for each commit-graph sub-command. > > > > > > The commit graph commands were in flux when trace2 was making it's way to > > > git. Now that we have enough sub-commands in commit-graph, we can label > > > the > > > various modes within them. Distinguishing between read, write and verify > > > is > > > a great start. > > > > > > Signed-off-by: Garima Singh garima.singh@microsoft.com > > > [garima.singh@microsoft.com] > > > > > > CC: jeffhost@microsoft.com, stolee@gmail.com, garimasigit@gmail.com, > > > avarab@gmail.com > > > > > > Garima Singh (1): > > > commit-graph: emit trace2 cmd_mode for each sub-command > > > > > > builtin/commit-graph.c | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > > > > base-commit: 745f6812895b31c02b29bdfe4ae8e5498f776c26 > > > Published-As: > > > https://github.com/gitgitgadget/git/releases/tag/pr-323%2Fgarimasi514%2FcoreGit-commit-graph-trace2-v1 > > > > > > Fetch-It-Via: git fetch https://github.com/gitgitgadget/git > > > pr-323/garimasi514/coreGit-commit-graph-trace2-v1 > > > Pull-Request: https://github.com/gitgitgadget/git/pull/323 > > > > > Looks good to me. Thanks! This was not yet picked up into `pu`, correct? Or did I miss anything? Ciao, Dscho > > Signed-off-by: Jeff Hostetler <jeffhost@microsoft.com> > > ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-10-02 8:03 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-08-27 16:56 [PATCH 0/1] commit-graph: emit trace2 cmd_mode for each sub-command Garima Singh via GitGitGadget 2019-08-27 16:56 ` [PATCH 1/1] " Garima Singh via GitGitGadget 2019-09-10 13:54 ` [PATCH 0/1] " Garima Singh 2019-09-11 18:17 ` Jeff Hostetler 2019-10-02 8:02 ` Johannes Schindelin
Code repositories for project(s) associated with this public inbox https://80x24.org/mirrors/git.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).