reftable/generic: drop interface
The `reftable_table` interface provides a generic infrastructure that can abstract away whether the underlying table is a single table, or a merged table. This abstraction can make it rather hard to reason about the code. We didn't ever use it to implement the reftable backend, and with the preceding patches in this patch series we in fact don't use it at all anymore. Furthermore, it became somewhat useless with the recent refactorings that made it possible to seek reftable iterators multiple times, as these now provide generic access to tables for us. The interface is thus redundant and only brings unnecessary complexity with it. Remove the `struct reftable_table` interface and its associated functions. Signed-off-by: Patrick Steinhardt <ps@pks.im> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Patrick Steinhardt committed
Aug 22, 2024 at 08:35 UTC
6014639837a8f118513563e6d9e3ff404e470b31
12 files changed
-245
Makefile
-1
@@ -2674,7 +2674,6 @@ REFTABLE_OBJS += reftable/merged.o
2674
REFTABLE_OBJS += reftable/pq.o
2675
REFTABLE_OBJS += reftable/reader.o
2676
REFTABLE_OBJS += reftable/record.o
2677
-REFTABLE_OBJS += reftable/generic.o
2677
REFTABLE_OBJS += reftable/stack.o
2678
REFTABLE_OBJS += reftable/tree.o
2679
REFTABLE_OBJS += reftable/writer.o
reftable/generic.c
deleted
-77
@@ -1,77 +0,0 @@
1
-/*
2
-Copyright 2020 Google LLC
3
-
4
-Use of this source code is governed by a BSD-style
5
-license that can be found in the LICENSE file or at
6
-https://developers.google.com/open-source/licenses/bsd
7
-*/
8
-
9
-#include "constants.h"
10
-#include "record.h"
11
-#include "generic.h"
12
-#include "iter.h"
13
-#include "reftable-iterator.h"
14
-#include "reftable-generic.h"
15
-
16
-void table_init_iter(struct reftable_table *tab,
17
- struct reftable_iterator *it,
18
- uint8_t typ)
19
-{
20
-
21
- tab->ops->init_iter(tab->table_arg, it, typ);
22
-}
23
-
24
-void reftable_table_init_ref_iter(struct reftable_table *tab,
25
- struct reftable_iterator *it)
26
-{
27
- table_init_iter(tab, it, BLOCK_TYPE_REF);
28
-}
29
-
30
-void reftable_table_init_log_iter(struct reftable_table *tab,
31
- struct reftable_iterator *it)
32
-{
33
- table_init_iter(tab, it, BLOCK_TYPE_LOG);
34
-}
35
-
36
-int reftable_table_read_ref(struct reftable_table *tab, const char *name,
37
- struct reftable_ref_record *ref)
38
-{
39
- struct reftable_iterator it = { NULL };
40
- int err;
41
-
42
- reftable_table_init_ref_iter(tab, &it);
43
-
44
- err = reftable_iterator_seek_ref(&it, name);
45
- if (err)
46
- goto done;
47
-
48
- err = reftable_iterator_next_ref(&it, ref);
49
- if (err)
50
- goto done;
51
-
52
- if (strcmp(ref->refname, name) ||
53
- reftable_ref_record_is_deletion(ref)) {
54
- reftable_ref_record_release(ref);
55
- err = 1;
56
- goto done;
57
- }
58
-
59
-done:
60
- reftable_iterator_destroy(&it);
61
- return err;
62
-}
63
-
64
-uint64_t reftable_table_max_update_index(struct reftable_table *tab)
65
-{
66
- return tab->ops->max_update_index(tab->table_arg);
67
-}
68
-
69
-uint64_t reftable_table_min_update_index(struct reftable_table *tab)
70
-{
71
- return tab->ops->min_update_index(tab->table_arg);
72
-}
73
-
74
-uint32_t reftable_table_hash_id(struct reftable_table *tab)
75
-{
76
- return tab->ops->hash_id(tab->table_arg);
77
-}
reftable/generic.h
deleted
-27
@@ -1,27 +0,0 @@
1
-/*
2
-Copyright 2020 Google LLC
3
-
4
-Use of this source code is governed by a BSD-style
5
-license that can be found in the LICENSE file or at
6
-https://developers.google.com/open-source/licenses/bsd
7
-*/
8
-
9
-#ifndef GENERIC_H
10
-#define GENERIC_H
11
-
12
-#include "record.h"
13
-#include "reftable-generic.h"
14
-
15
-/* generic interface to reftables */
16
-struct reftable_table_vtable {
17
- void (*init_iter)(void *tab, struct reftable_iterator *it, uint8_t typ);
18
- uint32_t (*hash_id)(void *tab);
19
- uint64_t (*min_update_index)(void *tab);
20
- uint64_t (*max_update_index)(void *tab);
21
-};
22
-
23
-void table_init_iter(struct reftable_table *tab,
24
- struct reftable_iterator *it,
25
- uint8_t typ);
26
-
27
-#endif
reftable/iter.c
-1
@@ -11,7 +11,6 @@ https://developers.google.com/open-source/licenses/bsd
11
#include "system.h"
12
13
#include "block.h"
14
-#include "generic.h"
14
#include "constants.h"
15
#include "reader.h"
16
#include "reftable-error.h"
reftable/iter.h
-1
@@ -14,7 +14,6 @@ https://developers.google.com/open-source/licenses/bsd
14
#include "record.h"
15
16
#include "reftable-iterator.h"
17
-#include "reftable-generic.h"
17
18
/*
19
* The virtual function table for implementing generic reftable iterators.
reftable/merged.c
-38
@@ -13,7 +13,6 @@ https://developers.google.com/open-source/licenses/bsd
13
#include "pq.h"
14
#include "reader.h"
15
#include "record.h"
16
-#include "generic.h"
16
#include "reftable-merged.h"
17
#include "reftable-error.h"
18
#include "system.h"
@@ -270,40 +269,3 @@ uint32_t reftable_merged_table_hash_id(struct reftable_merged_table *mt)
269
{
270
return mt->hash_id;
271
}
273
-
274
-static void reftable_merged_table_init_iter_void(void *tab,
275
- struct reftable_iterator *it,
276
- uint8_t typ)
277
-{
278
- merged_table_init_iter(tab, it, typ);
279
-}
280
-
281
-static uint32_t reftable_merged_table_hash_id_void(void *tab)
282
-{
283
- return reftable_merged_table_hash_id(tab);
284
-}
285
-
286
-static uint64_t reftable_merged_table_min_update_index_void(void *tab)
287
-{
288
- return reftable_merged_table_min_update_index(tab);
289
-}
290
-
291
-static uint64_t reftable_merged_table_max_update_index_void(void *tab)
292
-{
293
- return reftable_merged_table_max_update_index(tab);
294
-}
295
-
296
-static struct reftable_table_vtable merged_table_vtable = {
297
- .init_iter = reftable_merged_table_init_iter_void,
298
- .hash_id = reftable_merged_table_hash_id_void,
299
- .min_update_index = reftable_merged_table_min_update_index_void,
300
- .max_update_index = reftable_merged_table_max_update_index_void,
301
-};
302
-
303
-void reftable_table_from_merged_table(struct reftable_table *tab,
304
- struct reftable_merged_table *merged)
305
-{
306
- assert(!tab->ops);
307
- tab->ops = &merged_table_vtable;
308
- tab->table_arg = merged;
309
-}
reftable/reader.c
-41
@@ -11,11 +11,9 @@ https://developers.google.com/open-source/licenses/bsd
11
#include "system.h"
12
#include "block.h"
13
#include "constants.h"
14
-#include "generic.h"
14
#include "iter.h"
15
#include "record.h"
16
#include "reftable-error.h"
18
-#include "reftable-generic.h"
17
18
uint64_t block_source_size(struct reftable_block_source *source)
19
{
@@ -759,45 +757,6 @@ uint64_t reftable_reader_min_update_index(struct reftable_reader *r)
757
return r->min_update_index;
758
}
759
762
-/* generic table interface. */
763
-
764
-static void reftable_reader_init_iter_void(void *tab,
765
- struct reftable_iterator *it,
766
- uint8_t typ)
767
-{
768
- reader_init_iter(tab, it, typ);
769
-}
770
-
771
-static uint32_t reftable_reader_hash_id_void(void *tab)
772
-{
773
- return reftable_reader_hash_id(tab);
774
-}
775
-
776
-static uint64_t reftable_reader_min_update_index_void(void *tab)
777
-{
778
- return reftable_reader_min_update_index(tab);
779
-}
780
-
781
-static uint64_t reftable_reader_max_update_index_void(void *tab)
782
-{
783
- return reftable_reader_max_update_index(tab);
784
-}
785
-
786
-static struct reftable_table_vtable reader_vtable = {
787
- .init_iter = reftable_reader_init_iter_void,
788
- .hash_id = reftable_reader_hash_id_void,
789
- .min_update_index = reftable_reader_min_update_index_void,
790
- .max_update_index = reftable_reader_max_update_index_void,
791
-};
792
-
793
-void reftable_table_from_reader(struct reftable_table *tab,
794
- struct reftable_reader *reader)
795
-{
796
- assert(!tab->ops);
797
- tab->ops = &reader_vtable;
798
- tab->table_arg = reader;
799
-}
800
-
760
int reftable_reader_print_blocks(const char *tablename)
761
{
762
struct {
reftable/reftable-generic.h
deleted
-44
@@ -1,44 +0,0 @@
1
-/*
2
-Copyright 2020 Google LLC
3
-
4
-Use of this source code is governed by a BSD-style
5
-license that can be found in the LICENSE file or at
6
-https://developers.google.com/open-source/licenses/bsd
7
-*/
8
-
9
-#ifndef REFTABLE_GENERIC_H
10
-#define REFTABLE_GENERIC_H
11
-
12
-#include "reftable-iterator.h"
13
-
14
-struct reftable_table_vtable;
15
-
16
-/*
17
- * Provides a unified API for reading tables, either merged tables, or single
18
- * readers. */
19
-struct reftable_table {
20
- struct reftable_table_vtable *ops;
21
- void *table_arg;
22
-};
23
-
24
-void reftable_table_init_ref_iter(struct reftable_table *tab,
25
- struct reftable_iterator *it);
26
-
27
-void reftable_table_init_log_iter(struct reftable_table *tab,
28
- struct reftable_iterator *it);
29
-
30
-/* returns the hash ID from a generic reftable_table */
31
-uint32_t reftable_table_hash_id(struct reftable_table *tab);
32
-
33
-/* returns the max update_index covered by this table. */
34
-uint64_t reftable_table_max_update_index(struct reftable_table *tab);
35
-
36
-/* returns the min update_index covered by this table. */
37
-uint64_t reftable_table_min_update_index(struct reftable_table *tab);
38
-
39
-/* convenience function to read a single ref. Returns < 0 for error, 0
40
- for success, and 1 if ref not found. */
41
-int reftable_table_read_ref(struct reftable_table *tab, const char *name,
42
- struct reftable_ref_record *ref);
43
-
44
-#endif
reftable/reftable-merged.h
-6
@@ -26,8 +26,6 @@ https://developers.google.com/open-source/licenses/bsd
26
/* A merged table is implements seeking/iterating over a stack of tables. */
27
struct reftable_merged_table;
28
29
-/* A generic reftable; see below. */
30
-struct reftable_table;
29
struct reftable_reader;
30
31
/*
@@ -60,8 +58,4 @@ void reftable_merged_table_free(struct reftable_merged_table *m);
58
/* return the hash ID of the merged table. */
59
uint32_t reftable_merged_table_hash_id(struct reftable_merged_table *m);
60
63
-/* create a generic table from reftable_merged_table */
64
-void reftable_table_from_merged_table(struct reftable_table *tab,
65
- struct reftable_merged_table *table);
66
-
61
#endif
reftable/reftable-reader.h
-7
@@ -23,9 +23,6 @@
23
/* The reader struct is a handle to an open reftable file. */
24
struct reftable_reader;
25
26
-/* Generic table. */
27
-struct reftable_table;
28
-
26
/* reftable_new_reader opens a reftable for reading. If successful,
27
* returns 0 code and sets pp. The name is used for creating a
28
* stack. Typically, it is the basename of the file. The block source
@@ -60,10 +57,6 @@ uint64_t reftable_reader_max_update_index(struct reftable_reader *r);
57
/* return the min_update_index for a table */
58
uint64_t reftable_reader_min_update_index(struct reftable_reader *r);
59
63
-/* creates a generic table from a file reader. */
64
-void reftable_table_from_reader(struct reftable_table *tab,
65
- struct reftable_reader *reader);
66
-
60
/* print blocks onto stdout for debugging. */
61
int reftable_reader_print_blocks(const char *tablename);
62
reftable/stack.c
-1
@@ -14,7 +14,6 @@ https://developers.google.com/open-source/licenses/bsd
14
#include "merged.h"
15
#include "reader.h"
16
#include "reftable-error.h"
17
-#include "reftable-generic.h"
17
#include "reftable-record.h"
18
#include "reftable-merged.h"
19
#include "writer.h"
t/unit-tests/t-reftable-merged.c
-1
@@ -12,7 +12,6 @@ https://developers.google.com/open-source/licenses/bsd
12
#include "reftable/merged.h"
13
#include "reftable/reader.h"
14
#include "reftable/reftable-error.h"
15
-#include "reftable/reftable-generic.h"
15
#include "reftable/reftable-merged.h"
16
#include "reftable/reftable-writer.h"
17