Skip to content

Geo: Refactor replicables_for_geo_node method

Michael Kozono requested to merge mk/refactor-replicables-method into master

What does this MR do?

This MR is a refactor. No change in behavior.

The aim is to avoid ambiguity and confusion over the intent and usage of this method.

  • Removes unused node argument
    • because it is unused.
    • because the current implementations are secondary-specific, but it looks like you could pass in a primary node.
  • Renames the method from replicables_for_geo_node to replicables_for_current_secondary
    • because it is more specific (adds more context)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Michael Kozono

Merge request reports