Re: [PATCH RFC v3 0/3] Introduce git-last-modified(1) command

[Junio C Hamano <gitster@pobox.com>]

Toon Claes <toon@iotcl.com> writes:

> Again thanks to Taylor and the people at GitHub for sharing these
> patches. I hope we can work together to get this upstreamed.
>
> [1]: https://lore.kernel.org/git/patch-1.1-0ea849d900b-20230205T204104Z-avarab@gmail.com/
> [2]: https://lore.kernel.org/git/Z+XJ+1L3PnC9Dyba@nand.local/
> [3]: https://lore.kernel.org/git/20250326-toon-blame-tree-v1-3-4173133f3786@iotcl.com/
> [4]: git@github.com:ttaylorr/git.git
> [5]: https://lore.kernel.org/git/aCbBKj7O9LjO3SMK@pks.im/
>
> --
> Cheers,
> Toon
>
> Signed-off-by: Toon Claes <toon@iotcl.com>
> ---
> Changes in v3:
> - Updated benchmarks in commit messages.
> - Removed the patches that attempt to increase performance for tree
>   entries that have not been updated in a long time. (see above)
> - Move handling failure in `last_modified_init()` to the caller.
> - Sorted #include clauses lexicographically.
> - Removed unneeded `commit` in `struct last_modified_entry`.
> - Renamed some functions/variables and added some comments to make it
>   easier to understand.
> - Removed unnecessary checking of the commit-graph generation number.
> - Link to v2: https://lore.kernel.org/r/20250523-toon-new-blame-tree-v2-0-101e4ca4c1c9@iotcl.com
>
> Changes in v2:
> - The subcommand is renamed from `blame-tree` to `last-modified`
> - Documentation is added. Here we mark the command as experimental.
> - Some test cases are added related to merges.
> - Link to v1: https://lore.kernel.org/r/20250422-toon-new-blame-tree-v1-0-fdb51b8a394a@iotcl.com
>
> ---
> Toon Claes (3):
>       last-modified: new subcommand to show when files were last modified
>       t/perf: add last-modified perf script
>       last-modified: use Bloom filters when available
>
>  .gitignore                           |   1 +
>  Documentation/git-last-modified.adoc |  49 +++++++
>  Documentation/meson.build            |   1 +
>  Makefile                             |   2 +
>  builtin.h                            |   1 +
>  builtin/last-modified.c              |  44 ++++++
>  command-list.txt                     |   1 +
>  git.c                                |   1 +
>  last-modified.c                      | 257 +++++++++++++++++++++++++++++++++++
>  last-modified.h                      |  35 +++++
>  meson.build                          |   2 +
>  t/meson.build                        |   2 +
>  t/perf/p8020-last-modified.sh        |  21 +++
>  t/t8020-last-modified.sh             | 204 +++++++++++++++++++++++++++
>  14 files changed, 621 insertions(+)

FWI, "git last-modified -h" does not work; its output is expected to
match what is in "git help last-modified", and t0450 would not pass
without it.