c/m_dissemination: use chunked vector in update_leaders_request

The `update_leader_request` may only contain few of ntp leader updates.
In this case using a `fragmented_vector` with large chunk size is a
memory waste

Signed-off-by: Michał Maślanka <michal@redpanda.com>

src/v/cluster/metadata_dissemination_handler.cc

@@ -49,12 +49,12 @@ metadata_dissemination_handler::update_leadership_v2(
 
 ss::future<update_leadership_reply>
 metadata_dissemination_handler::do_update_leadership(
- fragmented_vector<ntp_leader_revision> leaders) {
+ chunked_vector<ntp_leader_revision> leaders) {
  vlog(clusterlog.trace, "Received a metadata update");
  co_await ss::parallel_for_each(
  boost::irange<ss::shard_id>(0, ss::smp::count),
  [this, leaders = std::move(leaders)](ss::shard_id shard) {
- fragmented_vector<ntp_leader_revision> local_leaders;
+ chunked_vector<ntp_leader_revision> local_leaders;
  local_leaders.reserve(leaders.size());
  std::copy(
  leaders.begin(), leaders.end(), std::back_inserter(local_leaders));

src/v/cluster/metadata_dissemination_handler.h

@@ -47,7 +47,7 @@ class metadata_dissemination_handler
 
 private:
  ss::future<update_leadership_reply>
- do_update_leadership(fragmented_vector<ntp_leader_revision>);
+ do_update_leadership(chunked_vector<ntp_leader_revision>);
 
  ss::sharded<partition_leaders_table>& _leaders;
 }; // namespace cluster

src/v/cluster/metadata_dissemination_service.cc

@@ -395,7 +395,7 @@ ss::future<> metadata_dissemination_service::update_leaders_with_health_report(
 ss::future<> metadata_dissemination_service::dispatch_one_update(
  model::node_id target_id, update_retry_meta& meta) {
  // copy updates to make retries possible
- fragmented_vector<ntp_leader_revision> updates;
+ chunked_vector<ntp_leader_revision> updates;
  updates.reserve(meta.updates.size());
  std::copy(
  meta.updates.begin(), meta.updates.end(), std::back_inserter(updates));

src/v/cluster/metadata_dissemination_types.h

@@ -104,7 +104,7 @@ struct update_leadership_request_v2
  serde::compat_version<0>> {
  using rpc_adl_exempt = std::true_type;
  static constexpr int8_t version = 0;
- fragmented_vector<ntp_leader_revision> leaders;
+ chunked_vector<ntp_leader_revision> leaders;
 
  update_leadership_request_v2() noexcept = default;
 
@@ -119,7 +119,7 @@ struct update_leadership_request_v2
  };
 
  update_leadership_request_v2 copy() const {
- fragmented_vector<ntp_leader_revision> leaders_cp;
+ chunked_vector<ntp_leader_revision> leaders_cp;
  leaders_cp.reserve(leaders.size());
  std::copy(
  leaders.begin(), leaders.end(), std::back_inserter(leaders_cp));
@@ -134,7 +134,7 @@ struct update_leadership_request_v2
  }
 
  explicit update_leadership_request_v2(
- fragmented_vector<ntp_leader_revision> leaders)
+ chunked_vector<ntp_leader_revision> leaders)
  : leaders(std::move(leaders)) {}
 
  auto serde_fields() { return std::tie(leaders); }