From: Calvin Wan <calvinwan@google.com>
To: John Cai <johncai86@gmail.com>, git@vger.kernel.org
Cc: Jonathan Tan <jonathantanmy@google.com>
Subject: Re: [PATCH] fetch —object-info-format: client option for object-info
Date: Thu, 10 Feb 2022 14:49:34 -0800 [thread overview]
Message-ID: <CAFySSZCZfekrdBH1NMArgPLdF4o1KQVY251BLyFgRxp=pDgEOA@mail.gmail.com> (raw)
In-Reply-To: <2B98F5EC-5F69-4DE1-89BF-33FCD1B03573@gmail.com>
> Maybe just --object-info and then provide a different flag for the <arguments> perhaps?
I agree. --object-info is a better name for the option. And with
<arguments>, there is also probably a better name -- will look into
the codebase and see what other options do.
> should object_info_format be cleared at some point?
I don't see why it needs to be cleared. The command only uses this
variable once in object-info.
Thank you for your input!
On Wed, Feb 9, 2022 at 11:09 AM John Cai <johncai86@gmail.com> wrote:
>
> Hi Calvin,
>
> On 8 Feb 2022, at 18:19, Calvin Wan wrote:
>
> > Add ‘—object-info-format’ option to fetch. This option allows
> > the client to make an object-info [1] command request to a server
> > that supports protocol v2.
> >
> > The transport implementation uses vtables [2], similar to how Git
> > fetches refs, to determine whether a process needs to be taken over
> > before sending the object-info request. Different protocols
> > require different setups for making requests.
> >
> > [1] https://lore.kernel.org/git/20210420233830.2181153-1-bga@google.com/
> > [2] https://lore.kernel.org/git/26f276956001a120cc9105b0071762c2fd4a45c5.1513287544.git.jonathantanmy@google.com/
> >
> > Helped-by: Jonathan Tan <jonathantanmy@google.com>
> > Signed-off-by: Calvin Wan <calvinwan@google.com>
> >
> > ---
> > builtin/fetch.c | 80 ++++++++++++++++++++----------
> > fetch-pack.c | 53 ++++++++++++++++++++
> > fetch-pack.h | 7 +++
> > t/t5583-fetch-object-info.sh | 95 ++++++++++++++++++++++++++++++++++++
> > transport-helper.c | 12 +++++
> > transport-internal.h | 1 +
> > transport.c | 63 ++++++++++++++++++++++++
> > transport.h | 1 +
> > 8 files changed, 286 insertions(+), 26 deletions(-)
> > create mode 100755 t/t5583-fetch-object-info.sh
> >
> > diff --git a/builtin/fetch.c b/builtin/fetch.c
> > index 5f06b21f8e..991063072b 100644
> > --- a/builtin/fetch.c
> > +++ b/builtin/fetch.c
> > @@ -29,6 +29,9 @@
> > #include "commit-graph.h"
> > #include "shallow.h"
> > #include "worktree.h"
> > +#include "protocol.h"
> > +#include "pkt-line.h"
> > +#include "connect.h"
> >
> > #define FORCED_UPDATES_DELAY_WARNING_IN_MS (10 * 1000)
> >
> > @@ -37,6 +40,7 @@ static const char * const builtin_fetch_usage[] = {
> > N_("git fetch [<options>] <group>"),
> > N_("git fetch --multiple [<options>] [(<repository> | <group>)...]"),
> > N_("git fetch --all [<options>]"),
> > + N_("git fetch --object-info-format=[<arguments>] <remote> [<object-ids>]"),
>
> --object-info-format seems to be more like a command, rather than a formatting
> option. When a flag ends with "format", it feels like it is specifying how to
> transform the output kind of like git for-each-ref --format. So --object-info-format looks
> feels like it's meant to transform the object-info output. Maybe just --object-info
> and then provide a different flag for the <arguments> perhaps?
>
> Would be interested in hearing what others think from a UI perspective.
>
> > NULL
> > };
> >
> > @@ -85,6 +89,7 @@ static struct string_list negotiation_tip = STRING_LIST_INIT_NODUP;
> > static int fetch_write_commit_graph = -1;
> > static int stdin_refspecs = 0;
> > static int negotiate_only;
> > +static struct string_list object_info_format = STRING_LIST_INIT_NODUP;
>
> should object_info_format be cleared at some point?
>
> >
> > static int git_fetch_config(const char *k, const char *v, void *cb)
> > {
> > @@ -220,6 +225,8 @@ static struct option builtin_fetch_options[] = {
> > N_("write the commit-graph after fetching")),
> > OPT_BOOL(0, "stdin", &stdin_refspecs,
> > N_("accept refspecs from stdin")),
> > + OPT_STRING_LIST(0, "object-info-format", &object_info_format, N_("option"),
> > + N_("command request arguments")),
> > OPT_END()
> > };
> >
> > @@ -2000,6 +2007,8 @@ int cmd_fetch(int argc, const char **argv, const char *prefix)
> > struct remote *remote = NULL;
> > int result = 0;
> > int prune_tags_ok = 1;
> > + struct oid_array oids = OID_ARRAY_INIT;
> > + struct object_id oid;
> >
> > packet_trace_identity("fetch");
> >
> > @@ -2019,6 +2028,7 @@ int cmd_fetch(int argc, const char **argv, const char *prefix)
> >
> > argc = parse_options(argc, argv, prefix,
> > builtin_fetch_options, builtin_fetch_usage, 0);
> > +
> > if (recurse_submodules != RECURSE_SUBMODULES_OFF) {
> > int *sfjc = submodule_fetch_jobs_config == -1
> > ? &submodule_fetch_jobs_config : NULL;
> > @@ -2057,34 +2067,52 @@ int cmd_fetch(int argc, const char **argv, const char *prefix)
> > if (dry_run)
> > write_fetch_head = 0;
> >
> > - if (all) {
> > - if (argc == 1)
> > - die(_("fetch --all does not take a repository argument"));
> > - else if (argc > 1)
> > - die(_("fetch --all does not make sense with refspecs"));
> > - (void) for_each_remote(get_one_remote_for_fetch, &list);
> > - } else if (argc == 0) {
> > - /* No arguments -- use default remote */
> > - remote = remote_get(NULL);
> > - } else if (multiple) {
> > - /* All arguments are assumed to be remotes or groups */
> > - for (i = 0; i < argc; i++)
> > - if (!add_remote_or_group(argv[i], &list))
> > - die(_("no such remote or remote group: %s"),
> > - argv[i]);
> > - } else {
> > - /* Single remote or group */
> > - (void) add_remote_or_group(argv[0], &list);
> > - if (list.nr > 1) {
> > - /* More than one remote */
> > - if (argc > 1)
> > - die(_("fetching a group and specifying refspecs does not make sense"));
> > + if (object_info_format.nr > 0) {
> > + if (argc == 0 || argc == 1) {
> > + die(_("must supply remote and object ids when using --object-info-format"));
> > } else {
> > - /* Zero or one remotes */
> > remote = remote_get(argv[0]);
> > - prune_tags_ok = (argc == 1);
> > - argc--;
> > - argv++;
> > + for (i = 1; i < argc; i++) {
> > + if (get_oid(argv[i], &oid))
> > + return error(_("malformed object name '%s'"), argv[i]);
> > + oid_array_append(&oids, &oid);
> > + }
> > + }
> > + gtransport = prepare_transport(remote, 0);
> > + gtransport->server_options = &object_info_format;
> > + result = transport_fetch_object_info(gtransport, &oids);
> > +
> > + return result;
> > + } else {
> > + if (all) {
> > + if (argc == 1)
> > + die(_("fetch --all does not take a repository argument"));
> > + else if (argc > 1)
> > + die(_("fetch --all does not make sense with refspecs"));
> > + (void) for_each_remote(get_one_remote_for_fetch, &list);
> > + } else if (argc == 0) {
> > + /* No arguments -- use default remote */
> > + remote = remote_get(NULL);
> > + } else if (multiple) {
> > + /* All arguments are assumed to be remotes or groups */
> > + for (i = 0; i < argc; i++)
> > + if (!add_remote_or_group(argv[i], &list))
> > + die(_("no such remote or remote group: %s"),
> > + argv[i]);
> > + } else {
> > + /* Single remote or group */
> > + (void) add_remote_or_group(argv[0], &list);
> > + if (list.nr > 1) {
> > + /* More than one remote */
> > + if (argc > 1)
> > + die(_("fetching a group and specifying refspecs does not make sense"));
> > + } else {
> > + /* Zero or one remotes */
> > + remote = remote_get(argv[0]);
> > + prune_tags_ok = (argc == 1);
> > + argc--;
> > + argv++;
> > + }
> > }
> > }
> >
> > diff --git a/fetch-pack.c b/fetch-pack.c
> > index dd6ec449f2..d1c5254aa8 100644
> > --- a/fetch-pack.c
> > +++ b/fetch-pack.c
> > @@ -1265,6 +1265,59 @@ static void write_fetch_command_and_capabilities(struct strbuf *req_buf,
> > packet_buf_delim(req_buf);
> > }
> >
> > +static void write_object_info_command_and_capabilities(struct strbuf *req_buf,
> > + const struct string_list *server_options)
> > +{
> > + const char *hash_name;
> > +
> > + if (server_supports_v2("object-info", 1))
> > + packet_buf_write(req_buf, "command=object-info");
> > + if (server_supports_v2("agent", 0))
> > + packet_buf_write(req_buf, "agent=%s", git_user_agent_sanitized());
> > + if (advertise_sid && server_supports_v2("session-id", 0))
> > + packet_buf_write(req_buf, "session-id=%s", trace2_session_id());
> > + if (server_options && server_options->nr &&
> > + server_supports_v2("server-option", 1)) {
> > + int i;
> > + for (i = 0; i < server_options->nr; i++)
> > + packet_buf_write(req_buf, "server-option=%s",
> > + server_options->items[i].string);
> > + }
> > +
> > + if (server_feature_v2("object-format", &hash_name)) {
> > + int hash_algo = hash_algo_by_name(hash_name);
> > + if (hash_algo_by_ptr(the_hash_algo) != hash_algo)
> > + die(_("mismatched algorithms: client %s; server %s"),
> > + the_hash_algo->name, hash_name);
> > + packet_buf_write(req_buf, "object-format=%s", the_hash_algo->name);
> > + } else if (hash_algo_by_ptr(the_hash_algo) != GIT_HASH_SHA1) {
> > + die(_("the server does not support algorithm '%s'"),
> > + the_hash_algo->name);
> > + }
> > + packet_buf_delim(req_buf);
> > +}
> > +
> > +void send_object_info_request(int fd_out, struct object_info_args *args)
> > +{
> > + struct strbuf req_buf = STRBUF_INIT;
> > + int i;
> > +
> > + write_object_info_command_and_capabilities(&req_buf, args->server_options);
> > +
> > + if (string_list_has_string(args->server_options, "size"))
> > + packet_buf_write(&req_buf, "size");
> > +
> > + for (i = 0; i < args->oids->nr; i++) {
> > + packet_buf_write(&req_buf, "oid %s\n", oid_to_hex(&args->oids->oid[i]));
> > + }
> > +
> > + packet_buf_flush(&req_buf);
> > + if (write_in_full(fd_out, req_buf.buf, req_buf.len) < 0)
> > + die_errno(_("unable to write request to remote"));
> > +
> > + strbuf_release(&req_buf);
> > +}
> > +
> > static int send_fetch_request(struct fetch_negotiator *negotiator, int fd_out,
> > struct fetch_pack_args *args,
> > const struct ref *wants, struct oidset *common,
> > diff --git a/fetch-pack.h b/fetch-pack.h
> > index 7f94a2a583..2ad5ec5c64 100644
> > --- a/fetch-pack.h
> > +++ b/fetch-pack.h
> > @@ -68,6 +68,11 @@ struct fetch_pack_args {
> > unsigned connectivity_checked:1;
> > };
> >
> > +struct object_info_args {
> > + const struct string_list *server_options;
> > + const struct oid_array *oids;
> > +};
> > +
> > /*
> > * sought represents remote references that should be updated from.
> > * On return, the names that were found on the remote will have been
> > @@ -101,4 +106,6 @@ void negotiate_using_fetch(const struct oid_array *negotiation_tips,
> > */
> > int report_unmatched_refs(struct ref **sought, int nr_sought);
> >
> > +void send_object_info_request(int fd_out, struct object_info_args *args);
> > +
> > #endif
> > diff --git a/t/t5583-fetch-object-info.sh b/t/t5583-fetch-object-info.sh
> > new file mode 100755
> > index 0000000000..93b09da3b1
> > --- /dev/null
> > +++ b/t/t5583-fetch-object-info.sh
> > @@ -0,0 +1,95 @@
> > +#!/bin/sh
> > +
> > +test_description='test git fetch object-info version 2'
> > +
> > +TEST_NO_CREATE_REPO=1
> > +
> > +GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
> > +export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
> > +
> > +. ./test-lib.sh
> > +
> > +initial_directory=$pwd
> > +
> > +# Test fetch object-info with 'git://' transport
> > +#
> > +. "$TEST_DIRECTORY"/lib-git-daemon.sh
> > +start_git_daemon --export-all --enable=receive-pack
> > +daemon_parent=$GIT_DAEMON_DOCUMENT_ROOT_PATH/parent
> > +
> > +
> > +test_expect_success 'create repo to be served by git-daemon' '
> > + git init "$daemon_parent" &&
> > + test_commit -C "$daemon_parent" message1 a.txt
> > +'
> > +
> > +test_expect_success 'fetch object-info with git:// using protocol v2' '
> > + test_when_finished "cd '$initial_directory'" &&
> > +
> > + cd "$daemon_parent" &&
> > +
> > + cat >actual <<-EOF &&
> > + $(GIT_TRACE_PACKET=1 git -c protocol.version=2 fetch --object-info-format=size "$GIT_DAEMON_URL/parent" $(git rev-parse message1:a.txt))
> > + EOF
> > +
> > + cat >expect <<-EOF &&
> > + $(git rev-parse message1:a.txt) $(wc -c <a.txt | xargs)
> > + EOF
> > +
> > + test_cmp expect actual
> > +'
> > +stop_git_daemon
> > +
> > +# Test protocol v2 with 'http://' transport
> > +#
> > +. "$TEST_DIRECTORY"/lib-httpd.sh
> > +start_httpd
> > +
> > +test_expect_success 'create repo to be served by http:// transport' '
> > + git init "$HTTPD_DOCUMENT_ROOT_PATH/http_parent" &&
> > + git -C "$HTTPD_DOCUMENT_ROOT_PATH/http_parent" config http.receivepack true &&
> > + test_commit -C "$HTTPD_DOCUMENT_ROOT_PATH/http_parent" message1 a.txt
> > +'
> > +
> > +test_expect_success 'fetch object-info with http:// using protocol v2' '
> > + test_when_finished "cd '$initial_directory'" &&
> > +
> > + cd "$HTTPD_DOCUMENT_ROOT_PATH/http_parent" &&
> > +
> > + cat >actual <<-EOF &&
> > + $(git -c protocol.version=2 fetch --object-info-format=size "$HTTPD_URL/smart/http_parent" $(git rev-parse message1:a.txt))
> > + EOF
> > +
> > + cat >expect <<-EOF &&
> > + $(git rev-parse message1:a.txt) $(wc -c <a.txt | xargs)
> > + EOF
> > +
> > + test_cmp expect actual
> > +'
> > +
> > +# Test fetch object-info with 'file://' transport
> > +#
> > +
> > +test_expect_success 'create repo to be serbed by file:// transport' '
> > + git init server &&
> > + test_commit -C server message1 a.txt &&
> > + git -C server config protocol.version 2
> > +'
> > +
> > +test_expect_success 'fetch object-info with file:// using protocol v2' '
> > + test_when_finished "cd '$initial_directory'" &&
> > +
> > + cd server &&
> > +
> > + cat >actual <<-EOF &&
> > + $(git fetch --object-info-format=size "file://$(pwd)" $(git rev-parse message1:a.txt))
> > + EOF
> > +
> > + cat >expect <<-EOF &&
> > + $(git rev-parse message1:a.txt) $(wc -c <a.txt | xargs)
> > + EOF
> > +
> > + test_cmp expect actual
> > +'
> > +
> > +test_done
> > \ No newline at end of file
> > diff --git a/transport-helper.c b/transport-helper.c
> > index a0297b0986..9ecda03dde 100644
> > --- a/transport-helper.c
> > +++ b/transport-helper.c
> > @@ -671,6 +671,17 @@ static int connect_helper(struct transport *transport, const char *name,
> > static struct ref *get_refs_list_using_list(struct transport *transport,
> > int for_push);
> >
> > +static int fetch_object_info(struct transport *transport, struct oid_array *oids)
> > +{
> > + get_helper(transport);
> > +
> > + if (process_connect(transport, 0)) {
> > + do_take_over(transport);
> > + return transport->vtable->fetch_object_info(transport, oids);
> > + }
> > + return -1;
> > +}
> > +
> > static int fetch_refs(struct transport *transport,
> > int nr_heads, struct ref **to_fetch)
> > {
> > @@ -1269,6 +1280,7 @@ static struct transport_vtable vtable = {
> > .get_refs_list = get_refs_list,
> > .fetch_refs = fetch_refs,
> > .push_refs = push_refs,
> > + .fetch_object_info = fetch_object_info,
> > .connect = connect_helper,
> > .disconnect = release_helper
> > };
> > diff --git a/transport-internal.h b/transport-internal.h
> > index c4ca0b733a..04fa015011 100644
> > --- a/transport-internal.h
> > +++ b/transport-internal.h
> > @@ -59,6 +59,7 @@ struct transport_vtable {
> > * use. disconnect() releases these resources.
> > **/
> > int (*disconnect)(struct transport *connection);
> > + int (*fetch_object_info)(struct transport *transport, struct oid_array *oids);
> > };
> >
> > #endif
> > diff --git a/transport.c b/transport.c
> > index 2a3e324154..0e1bf3c4cd 100644
> > --- a/transport.c
> > +++ b/transport.c
> > @@ -445,6 +445,62 @@ static int fetch_refs_via_pack(struct transport *transport,
> > return ret;
> > }
> >
> > +static int fetch_object_info(struct transport *transport, struct oid_array *oids)
> > +{
> > + int ret = 0;
> > + struct git_transport_data *data = transport->data;
> > + struct object_info_args args;
> > + struct packet_reader reader;
> > +
> > + memset(&args, 0, sizeof(args));
> > + args.server_options = transport->server_options;
> > + args.oids = oids;
> > +
> > + connect_setup(transport, 0);
> > + packet_reader_init(&reader, data->fd[0], NULL, 0,
> > + PACKET_READ_CHOMP_NEWLINE |
> > + PACKET_READ_GENTLE_ON_EOF |
> > + PACKET_READ_DIE_ON_ERR_PACKET);
> > + data->version = discover_version(&reader);
> > +
> > + if (data->version == protocol_unknown_version)
> > + BUG("unknown protocol version");
> > + else if (data->version <= protocol_v1)
> > + die_if_server_options(transport);
> > +
> > + switch (data->version) {
> > + case protocol_v2:
> > + send_object_info_request(data->fd[1], &args);
> > + break;
> > + case protocol_v1:
> > + case protocol_v0:
> > + die(_("wrong protocol version. expected v2"));
> > + case protocol_unknown_version:
> > + BUG("unknown protocol version");
> > + }
> > +
> > + if (packet_reader_read(&reader) != PACKET_READ_NORMAL) {
> > + die(_("error reading object info header"));
> > + }
> > + if (strcmp(reader.line, "size")) {
> > + die(_("expected 'size', received '%s'"), reader.line);
> > + }
> > + while (packet_reader_read(&reader) == PACKET_READ_NORMAL) {
> > + printf("%s\n", reader.line);
> > + }
> > +
> > + close(data->fd[0]);
> > +
> > + if (data->fd[1] >= 0) {
> > + close(data->fd[1]);
> > + }
> > + if (finish_connect(data->conn))
> > + ret = -1;
> > + data->conn = NULL;
> > +
> > + return ret;
> > +}
> > +
> > static int push_had_errors(struct ref *ref)
> > {
> > for (; ref; ref = ref->next) {
> > @@ -890,6 +946,7 @@ static struct transport_vtable taken_over_vtable = {
> > .get_refs_list = get_refs_via_connect,
> > .fetch_refs = fetch_refs_via_pack,
> > .push_refs = git_transport_push,
> > + .fetch_object_info = fetch_object_info,
> > .disconnect = disconnect_git
> > };
> >
> > @@ -1043,6 +1100,7 @@ static struct transport_vtable builtin_smart_vtable = {
> > .get_refs_list = get_refs_via_connect,
> > .fetch_refs = fetch_refs_via_pack,
> > .push_refs = git_transport_push,
> > + .fetch_object_info = fetch_object_info,
> > .connect = connect_git,
> > .disconnect = disconnect_git
> > };
> > @@ -1420,6 +1478,11 @@ const struct ref *transport_get_remote_refs(struct transport *transport,
> > return transport->remote_refs;
> > }
> >
> > +int transport_fetch_object_info(struct transport *transport, struct oid_array *oids) {
> > + transport->vtable->fetch_object_info(transport, oids);
> > + return 1;
> > +}
> > +
> > int transport_fetch_refs(struct transport *transport, struct ref *refs)
> > {
> > int rc;
> > diff --git a/transport.h b/transport.h
> > index 3f16e50c19..1c807591de 100644
> > --- a/transport.h
> > +++ b/transport.h
> > @@ -278,6 +278,7 @@ const struct ref *transport_get_remote_refs(struct transport *transport,
> > * This can only be called after fetching the remote refs.
> > */
> > const struct git_hash_algo *transport_get_hash_algo(struct transport *transport);
> > +int transport_fetch_object_info(struct transport *transport, struct oid_array *oids);
> > int transport_fetch_refs(struct transport *transport, struct ref *refs);
> >
> > /*
> >
> > base-commit: b23dac905bde28da47543484320db16312c87551
> > --
> > 2.33.0.664.g0785eb7698
prev parent reply other threads:[~2022-02-10 22:51 UTC|newest]
Thread overview: 103+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-08 23:19 [PATCH] fetch —object-info-format: client option for object-info Calvin Wan
2022-02-08 23:56 ` [PATCH v2] " Calvin Wan
2022-02-09 12:48 ` Philip Oakley
2022-02-10 22:32 ` Calvin Wan
2022-02-09 20:41 ` [PATCH v2] fetch object-info-format: " Jonathan Tan
2022-02-10 22:58 ` Calvin Wan
2022-03-28 19:11 ` [PATCH v3 0/3] object-info: add option for retrieving object info Calvin Wan
2022-03-28 19:11 ` [PATCH v3 1/3] fetch-pack: refactor packet writing and fetch options Calvin Wan
2022-03-29 22:54 ` Junio C Hamano
2022-03-29 23:01 ` Taylor Blau
2022-03-30 21:55 ` Jonathan Tan
2022-03-28 19:11 ` [PATCH v3 2/3] transfer.advertiseObjectInfo: add object-info config Calvin Wan
2022-03-29 22:34 ` Junio C Hamano
2022-03-29 22:48 ` Calvin Wan
2022-03-29 23:09 ` Taylor Blau
2022-03-28 19:11 ` [PATCH v3 3/3] object-info: add option for retrieving object info Calvin Wan
2022-03-29 19:57 ` Junio C Hamano
2022-03-29 22:54 ` Junio C Hamano
2022-03-30 0:49 ` Junio C Hamano
2022-03-30 22:31 ` Calvin Wan
2022-03-30 22:43 ` Jonathan Tan
2022-03-30 23:42 ` Junio C Hamano
2022-03-29 23:19 ` Taylor Blau
2022-03-30 22:47 ` Calvin Wan
2022-03-30 22:06 ` John Cai
2022-03-31 19:56 ` Calvin Wan
2022-04-01 16:16 ` Junio C Hamano
2022-03-30 22:07 ` Jonathan Tan
2022-03-30 22:12 ` Josh Steadmon
2022-03-30 22:46 ` Calvin Wan
2022-03-29 20:35 ` [PATCH v3 0/3] " Junio C Hamano
2022-03-29 22:40 ` Calvin Wan
2022-03-31 1:50 ` Junio C Hamano
2022-05-02 17:08 ` [PATCH v4 0/8] cat-file: add --batch-command remote-object-info command Calvin Wan
2022-05-02 17:08 ` [PATCH v4 1/8] fetch-pack: refactor packet writing Calvin Wan
2022-05-02 17:08 ` [PATCH v4 2/8] fetch-pack: move fetch default settings Calvin Wan
2022-05-02 22:58 ` Junio C Hamano
2022-05-03 23:06 ` Jonathan Tan
2022-05-05 18:13 ` Calvin Wan
2022-05-02 17:08 ` [PATCH v4 3/8] object-store: add function to free object_info contents Calvin Wan
2022-05-02 23:23 ` Junio C Hamano
2022-05-04 19:09 ` Junio C Hamano
2022-05-05 0:15 ` Junio C Hamano
2022-05-05 16:47 ` Calvin Wan
2022-05-05 17:01 ` Junio C Hamano
2022-05-02 17:09 ` [PATCH v4 4/8] object-info: send attribute packet regardless of object ids Calvin Wan
2022-05-03 0:05 ` Junio C Hamano
2022-05-03 19:21 ` Calvin Wan
2022-05-03 23:11 ` Jonathan Tan
2022-05-02 17:09 ` [PATCH v4 5/8] transport: add client side capability to request object-info Calvin Wan
2022-05-03 0:54 ` Junio C Hamano
2022-05-03 18:58 ` Calvin Wan
2022-05-04 15:42 ` Junio C Hamano
2022-05-03 23:15 ` Jonathan Tan
2022-05-04 15:50 ` Junio C Hamano
2022-05-02 17:09 ` [PATCH v4 6/8] transport: add object-info fallback to fetch Calvin Wan
2022-05-03 23:27 ` Jonathan Tan
2022-05-02 17:09 ` [PATCH v4 7/8] cat-file: move parse_cmd and DEFAULT_FORMAT up Calvin Wan
2022-05-02 17:09 ` [PATCH v4 8/8] cat-file: add --batch-command remote-object-info command Calvin Wan
2022-05-04 21:27 ` Jonathan Tan
2022-05-05 18:13 ` Calvin Wan
2022-05-05 18:44 ` Junio C Hamano
2022-05-05 19:09 ` Junio C Hamano
2022-05-05 19:19 ` Calvin Wan
2022-07-31 15:24 ` ZheNing Hu
2022-08-08 17:43 ` Calvin Wan
2022-07-28 23:02 ` [PATCH v5 0/6] " Calvin Wan
2022-07-28 23:56 ` Junio C Hamano
2022-07-29 0:02 ` Junio C Hamano
2022-07-31 8:41 ` Phillip Wood
2022-08-04 22:57 ` Calvin Wan
2022-09-30 23:23 ` Junio C Hamano
2022-07-28 23:02 ` [PATCH v5 1/6] fetch-pack: refactor packet writing Calvin Wan
2022-07-28 23:02 ` [PATCH v5 2/6] fetch-pack: move fetch initialization Calvin Wan
2022-07-28 23:02 ` [PATCH v5 3/6] protocol-caps: initialization bug fix Calvin Wan
2022-07-29 17:51 ` Junio C Hamano
2022-07-28 23:02 ` [PATCH v5 4/6] serve: advertise object-info feature Calvin Wan
2022-07-29 17:57 ` Junio C Hamano
2022-08-01 18:28 ` Calvin Wan
2022-08-01 18:44 ` Ævar Arnfjörð Bjarmason
2022-08-01 18:47 ` Junio C Hamano
2022-08-01 18:58 ` Calvin Wan
2022-07-28 23:02 ` [PATCH v5 5/6] transport: add client support for object-info Calvin Wan
2022-07-29 18:06 ` Junio C Hamano
2022-08-04 20:28 ` Calvin Wan
2022-08-01 13:30 ` Phillip Wood
2022-08-04 22:20 ` Calvin Wan
2022-08-08 10:07 ` Phillip Wood
2022-08-01 14:26 ` Phillip Wood
2022-08-08 9:16 ` Phillip Wood
2022-07-28 23:02 ` [PATCH v5 6/6] cat-file: add remote-object-info to batch-command Calvin Wan
2022-07-29 6:25 ` Ævar Arnfjörð Bjarmason
2022-08-07 5:50 ` ZheNing Hu
2022-08-08 18:07 ` Calvin Wan
2022-08-11 10:58 ` Ævar Arnfjörð Bjarmason
2022-07-29 18:21 ` Junio C Hamano
2022-08-08 18:37 ` Calvin Wan
2022-09-28 13:12 ` Ævar Arnfjörð Bjarmason
2022-07-31 15:02 ` [PATCH v4 0/8] cat-file: add --batch-command remote-object-info command ZheNing Hu
2022-08-08 17:32 ` Calvin Wan
2022-08-13 22:17 ` Junio C Hamano
2022-02-09 19:09 ` [PATCH] fetch —object-info-format: client option for object-info John Cai
2022-02-10 22:49 ` Calvin Wan [this message]
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='CAFySSZCZfekrdBH1NMArgPLdF4o1KQVY251BLyFgRxp=pDgEOA@mail.gmail.com' \
--to=calvinwan@google.com \
--cc=git@vger.kernel.org \
--cc=johncai86@gmail.com \
--cc=jonathantanmy@google.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).