From: Nathaniel Filardo <nwf20@cl.cam.ac.uk>
To: git@vger.kernel.org
Cc: Derrick Stolee <dstolee@microsoft.com>,
Nathaniel Filardo <nwf20@cl.cam.ac.uk>
Subject: [PATCHv2 1/5] count-objects: report statistics about kept packs
Date: Sat, 16 Mar 2019 20:05:16 +0000 [thread overview]
Message-ID: <20190316200520.14260-2-nwf20@cl.cam.ac.uk> (raw)
In-Reply-To: <20190316200520.14260-1-nwf20@cl.cam.ac.uk>
Specifically: number of kept packs, size of kept packs (and indexes),
and number of objects in kept packs.
Add documentation of resulting fields. While here, correct the omission
from ae72f68541 ("count-objects -v: show number of packs as well.",
2006-12-27) of the "packs" field.
Signed-off-by: Nathaniel Filardo <nwf20@cl.cam.ac.uk>
---
Documentation/git-count-objects.txt | 8 ++++++++
builtin/count-objects.c | 17 +++++++++++++++--
2 files changed, 23 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-count-objects.txt b/Documentation/git-count-objects.txt
index cb9b4d2e46..8d88f87856 100644
--- a/Documentation/git-count-objects.txt
+++ b/Documentation/git-count-objects.txt
@@ -26,10 +26,18 @@ count: the number of loose objects
+
size: disk space consumed by loose objects, in KiB (unless -H is specified)
+
+packs: the number of pack files
++
+packs-keep: the number of pack files marked kept
++
in-pack: the number of in-pack objects
+
+in-pack-keep: the number of objects in packs marked kept
++
size-pack: disk space consumed by the packs, in KiB (unless -H is specified)
+
+size-pack-keep: disk space consumed by kept packs, in KiB (unless -H is specified)
++
prune-packable: the number of loose objects that are also present in
the packs. These objects could be pruned using `git prune-packed`.
+
diff --git a/builtin/count-objects.c b/builtin/count-objects.c
index 3fae474f6f..0309c7907f 100644
--- a/builtin/count-objects.c
+++ b/builtin/count-objects.c
@@ -117,17 +117,24 @@ int cmd_count_objects(int argc, const char **argv, const char *prefix)
if (verbose) {
struct packed_git *p;
- unsigned long num_pack = 0;
- off_t size_pack = 0;
+ unsigned long num_pack = 0, num_pack_keep = 0;
+ unsigned long packed_keep = 0;
+ off_t size_pack = 0, size_pack_keep = 0;
struct strbuf loose_buf = STRBUF_INIT;
struct strbuf pack_buf = STRBUF_INIT;
struct strbuf garbage_buf = STRBUF_INIT;
+ struct strbuf pack_keep_buf = STRBUF_INIT;
for (p = get_all_packs(the_repository); p; p = p->next) {
if (!p->pack_local)
continue;
if (open_pack_index(p))
continue;
+ if (p->pack_keep || p->pack_keep_in_core) {
+ packed_keep += p->num_objects;
+ size_pack_keep += p->pack_size + p->index_size;
+ num_pack_keep++;
+ }
packed += p->num_objects;
size_pack += p->pack_size + p->index_size;
num_pack++;
@@ -136,12 +143,15 @@ int cmd_count_objects(int argc, const char **argv, const char *prefix)
if (human_readable) {
strbuf_humanise_bytes(&loose_buf, loose_size);
strbuf_humanise_bytes(&pack_buf, size_pack);
+ strbuf_humanise_bytes(&pack_keep_buf, size_pack_keep);
strbuf_humanise_bytes(&garbage_buf, size_garbage);
} else {
strbuf_addf(&loose_buf, "%lu",
(unsigned long)(loose_size / 1024));
strbuf_addf(&pack_buf, "%lu",
(unsigned long)(size_pack / 1024));
+ strbuf_addf(&pack_keep_buf, "%lu",
+ (unsigned long)(size_pack_keep / 1024));
strbuf_addf(&garbage_buf, "%lu",
(unsigned long)(size_garbage / 1024));
}
@@ -149,8 +159,11 @@ int cmd_count_objects(int argc, const char **argv, const char *prefix)
printf("count: %lu\n", loose);
printf("size: %s\n", loose_buf.buf);
printf("in-pack: %lu\n", packed);
+ printf("in-pack-keep: %lu\n", packed_keep);
printf("packs: %lu\n", num_pack);
+ printf("packs-keep: %lu\n", num_pack_keep);
printf("size-pack: %s\n", pack_buf.buf);
+ printf("size-pack-keep: %s\n", pack_keep_buf.buf);
printf("prune-packable: %lu\n", packed_loose);
printf("garbage: %lu\n", garbage);
printf("size-garbage: %s\n", garbage_buf.buf);
--
2.17.1
next prev parent reply other threads:[~2019-03-16 20:05 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-16 20:05 [PATCHv2 0/5] Speed up repacking when lots of pack-kept objects Nathaniel Filardo
2019-03-16 20:05 ` Nathaniel Filardo [this message]
2019-03-16 20:05 ` [PATCHv2 2/5] revision walk: optionally use sparse reachability Nathaniel Filardo
2019-03-16 20:05 ` [PATCHv2 3/5] repack: add --sparse and pass to pack-objects Nathaniel Filardo
2019-03-16 20:05 ` [PATCHv2 4/5] repack: optionally assume transitive kept packs Nathaniel Filardo
2019-03-16 20:05 ` [PATCHv2 5/5] builtin/gc: add --assume-pack-keep-transitive Nathaniel Filardo
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=20190316200520.14260-2-nwf20@cl.cam.ac.uk \
--to=nwf20@cl.cam.ac.uk \
--cc=dstolee@microsoft.com \
--cc=git@vger.kernel.org \
/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).