git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: jh/trace2-sid-fix, was Re: What's cooking in git.git (Apr 2019, #02; Wed, 10)
Date: Thu, 11 Apr 2019 00:32:55 +0200 (DST)	[thread overview]
Message-ID: <nycvar.QRO.7.76.6.1904110016030.41@tvgsbejvaqbjf.bet> (raw)
In-Reply-To: <xmqqr2ab2gs6.fsf@gitster-ct.c.googlers.com>

Hi Junio,

On Wed, 10 Apr 2019, Junio C Hamano wrote:

> * jh/trace2-sid-fix (2019-04-01) 7 commits
>  - trace2: make SIDs more unique
>  - trace2: clarify UTC datetime formatting
>  - trace2: report peak memory usage of the process
>  - trace2: use system config for default trace2 settings
>  - trace2: find exec-dir before trace2 initialization
>  - trace2: add absolute elapsed time to start event
>  - trace2: refactor setting process starting time
>
>  Polishing of the new trace2 facility continues.  The system-level
>  configuration can specify site-wide trace2 settings (which would be
>  loved by big-brother types ;-).

While this is all fun and joy to perpetuate the stereotypes, I think that
more people would potentially be interested in using this in their
development teams, if only they knew what the actual purpose of trace2 is
(which this comment does not describe well).

As you probably use this description for the release notes, maybe it would
be a good idea to replace the Orwell reference by something like this:

	This allows trace2 to be enabled in a site-wide configuration. The
	intended audience for this feature are development teams which
	want to analyze and identify performance bottlenecks and other
	problems typical in their common workflows.

If you still need to be convinced that this kind of setting can help
tremendously with improving Git *itself* to become faster, watch the talk
by John Briggs at GitMerge 2019:
https://www.youtube.com/watch?v=vat97a8C0o0

I cannot stress how crucial it is for our in-house use to have this kind
of detailed logging to steer our development efforts.

And obviously you are totally allowed to continue to make these jokes
about surveillance, at least as long as you admit that you know that
nobody wants to enable this outside their own development teams.

It would just be nice to see it ackowledged from time to time that these
analyses that we perform ("we" meaning the team in which I am embedded,
and other teams, e.g. within Google) have a direct benefit to the Git
project, as we no longer have to guess or surmise where our time is best
spent improving Git's performance, but we can focus our attention wisely
based on scientifically-sound statistics.

Ciao,
Dscho

>  Getting closer but still being discussed.
>  cf. <20190403000032.GA190454@google.com>

  parent reply	other threads:[~2019-04-10 22:33 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-09 18:08 What's cooking in git.git (Apr 2019, #02; Wed, 10) Junio C Hamano
2019-04-09 18:22 ` Denton Liu
2019-04-10  3:09   ` Junio C Hamano
2019-04-09 19:18 ` Thomas Gummerer
2019-04-10  3:01   ` Junio C Hamano
2019-04-10 20:45     ` Thomas Gummerer
2019-04-10 22:13     ` Johannes Schindelin
2019-04-10  0:46 ` Todd Zullinger
2019-04-10 10:12 ` Phillip Wood
2019-04-10 22:15   ` Johannes Schindelin
2019-04-10 18:29 ` Mazo, Andrey
2019-04-10 21:51 ` nd/switch-and-restore, was " Johannes Schindelin
2019-04-10 21:56 ` dl/warn-tagging-a-tag, " Johannes Schindelin
2019-04-12  1:51   ` Junio C Hamano
2019-04-10 22:32 ` Johannes Schindelin [this message]
2019-04-11 15:06 ` Christian Couder
2019-04-14  3:55   ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=nycvar.QRO.7.76.6.1904110016030.41@tvgsbejvaqbjf.bet \
    --to=johannes.schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).