[RATIS-1742] Remove slf4j-log4j12 test dependency from ratis-common and ratis-server

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0, 2.5.0
• Component/s: test
• Assignee: Tsz-wo Sze
• Reporter: Tsz-wo Sze
• Created: 12/Nov/22 23:43
• Updated: 27/Apr/23 17:53
• Resolved: 18/Nov/22 02:38

Description

For ratis-server, slf4j-log4j12 is unused so that it can be removed directly.

For ratis-common, slf4j-log4j12 is only used in Slf4jUtils after RATIS-1728. We may use reflection to implement the setLevel method.

We also fix the following test problems:

• There is a "multiple SLF4J bindings" warning so ratis-test should exclude the slf4j-simple from ratis-shell.
• The following deprecation warnings.

  [WARNING] /Users/szetszwo/ratis/apache-ratis/ratis-test/src/test/java/org/apache/ratis/server/raftlog/segmented/TestSegmentedRaftLog.java:[715,27] [deprecation] append(LogEntryProto...) in RaftLogSequentialOps has been deprecated
  [WARNING] /Users/szetszwo/ratis/apache-ratis/ratis-test/src/test/java/org/apache/ratis/datastream/TestDataStreamDisabled.java:[35,62] [deprecation] none() in ExpectedException has been deprecated
  

Issue Links

• relates to: RATIS-1728 Avoid org.apache.log4j imports
• links to: GitHub Pull Request #781