reftable/stack: add mechanism to notify callers on reload

Reftable stacks are reloaded in two cases:

  - When calling `reftable_stack_reload()`, if the stat-cache tells us
    that the stack has been modified.

  - When committing a reftable addition.

While callers can figure out the second case, they do not have a
mechanism to figure out whether `reftable_stack_reload()` led to an
actual reload of the on-disk data. All they can do is thus to assume
that data is always being reloaded in that case.

Improve the situation by introducing a new `on_reload()` callback to the
reftable options. If provided, the function will be invoked every time
the stack has indeed been reloaded. This allows callers to invalidate
data that depends on the current stack data.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/reftable/reftable-writer.h b/reftable/reftable-writer.h
index c85ef5a5bd14595d75f99457fef4407040e197c5..5f9afa620bb00de66c311765fb0ae8c6f56401ae 100644 (file)
--- a/reftable/reftable-writer.h
+++ b/reftable/reftable-writer.h
@@ -68,6 +68,15 @@ struct reftable_write_options {
         * fsync(3P) when unset.
         */
        int (*fsync)(int fd);
+
+       /*
+        * Callback function to execute whenever the stack is being reloaded.
+        * This can be used e.g. to discard cached information that relies on
+        * the old stack's data. The payload data will be passed as argument to
+        * the callback.
+        */
+       void (*on_reload)(void *payload);
+       void *on_reload_payload;
 };
 
 /* reftable_block_stats holds statistics for a single block type */

diff --git a/reftable/stack.c b/reftable/stack.c
index 8beb5c0541b70524b69cf3e2167f099808822902..59fd695a12c2033ed589a21ef1c9155eeecc4641 100644 (file)
--- a/reftable/stack.c
+++ b/reftable/stack.c
@@ -548,6 +548,10 @@ out:
                close(fd);
        free_names(names);
        free_names(names_after);
+
+       if (st->opts.on_reload)
+               st->opts.on_reload(st->opts.on_reload_payload);
+
        return err;
 }