From e78e64a92087b34b1078ade3c53f88e8739159fd Mon Sep 17 00:00:00 2001 From: Pavel Emelyanov Date: Tue, 28 Mar 2023 12:03:18 +0300 Subject: [PATCH] table: Carry v.u.generator down to populate_views() The method is called by view_builder::consumer when building a view and the consumer already has stable dependency reference on the view updates generator. Signed-off-by: Pavel Emelyanov --- db/view/view.cc | 2 +- replica/database.hh | 1 + replica/table.cc | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/db/view/view.cc b/db/view/view.cc index 035825c285..a0ac8255ca 100644 --- a/db/view/view.cc +++ b/db/view/view.cc @@ -2295,7 +2295,6 @@ public: if (!step.current_key.key().is_empty(*_step.reader.schema())) { load_views_to_build(); } - (void)_gen; } void load_views_to_build() { @@ -2396,6 +2395,7 @@ public: auto close_reader = defer([&reader] { reader.close().get(); }); reader.upgrade_schema(base_schema); _step.base->populate_views( + _gen, std::move(views), _step.current_token(), std::move(reader), diff --git a/replica/database.hh b/replica/database.hh index a10b178924..f3cfc62c16 100644 --- a/replica/database.hh +++ b/replica/database.hh @@ -1051,6 +1051,7 @@ public: // Reader's schema must be the same as the base schema of each of the views. future<> populate_views( + shared_ptr gen, std::vector, dht::token base_token, flat_mutation_reader_v2&&, diff --git a/replica/table.cc b/replica/table.cc index b7cffd1268..67498f2dfc 100644 --- a/replica/table.cc +++ b/replica/table.cc @@ -2107,6 +2107,7 @@ table::local_base_lock( * @return a future that resolves when the updates have been acknowledged by the view replicas */ future<> table::populate_views( + shared_ptr gen, std::vector views, dht::token base_token, flat_mutation_reader_v2&& reader,