Improve AbstractLeader class hierarchy 69/115969/2
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 21 Mar 2025 16:29:03 +0000 (17:29 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 21 Mar 2025 16:40:05 +0000 (17:40 +0100)
commit6fbaf5647c8ad3f6476b80dc0ae8f2b11508de89
tree6335a490f716869d5822ac45cdd29a6a74176e1d
parentf9c219a53e5192807bf850c726b22da29e3358c6
Improve AbstractLeader class hierarchy

AbstractLeader.handleAppendEntriesReply() always results in 'this' being
returned and is overridden in the three subclasses.

Rename it to processAppendEntriesReply(), without the ability to change
behavior and make it final. The subclasses then use it as a common
utility, doing their own thing as needed.

Since Leader is subclassed in tests, we lock down all its methods except
the single one that is being overridden.

Change-Id: I594bebedfa612f7e946d97040ea55c053c2c9f3d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/AbstractLeader.java
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/IsolatedLeader.java
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/Leader.java
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/PreLeader.java