Refactor: scheduler: de-functionize compatible_replica_for_node()

Both its only caller, compatible_replica(), and that function's only caller
logged the same information. Without the redundant log messages, it's just a
couple of lines.