Remove dead code

[eddyashton]

No description provided.

[Copilot]

Pull request overview

Removes an unused “removal” block in AFT Raft’s node-state maintenance logic, reducing dead code in the consensus implementation.

Changes:

• Delete an unused to_remove collection and associated loop in create_and_remove_node_state().

Comments suppressed due to low confidence (1)

src/consensus/aft/raft.h:2813

• create_and_remove_node_state() no longer contains any removal logic (the only removal-related block was dead). The name is now misleading; consider renaming it (and updating call sites) to reflect its actual behavior, or implement the intended removal of nodes not present in any active configuration to match the name.

      // Add all active nodes that are not already present in the node state.
      for (auto node_info : active_nodes)
      {
        if (node_info.first == state->node_id)
        {