[CASSANDRA-16855] Replace minor use of `json-simple` with Jackson

• Type: Improvement
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 5.0-alpha1, 5.0
• Component/s: Dependencies, Local/Other, Tool/nodetool
• Labels: pull-request-available
• Change Category: Code Clarity
• Complexity: Normal
• Platform: All

Source Control Link

https://github.com/apache/cassandra/commit/73da05f83ba2547662e6320cb2cb3576bf82c15f

Test and Documentation Plan

run CI

• Assignee: Stefan Miklosovic
• Reporter: Tatu Saloranta

Authors

Stefan Miklosovic

Reviewers

Brandon Williams, Ekaterina Dimitrova

• Created: 14/Aug/21 00:13
• Updated: 12/Sep/23 13:03
• Resolved: 02/May/23 20:08

Description

Jackson library is used for most JSON reading/writing, but there are couple of places where older "json-simple" library is used, mostly for diagnostics output. Replacing those minor usages would allow removal of a dependency, one for which the last release was made in 2012.

Places where json-simple is used are:

• src/java/org/apache/cassandra/db/ColumnFamilyStore.java
• src/java/org/apache/cassandra/db/commitlog/CommitLogDescriptor.java
• src/java/org/apache/cassandra/hints/HintsDescriptor.java
• src/java/org/apache/cassandra/tools/nodetool/stats/StatsPrinter.java

(and some matching usage in couple of test classes)

I can take a stab at replacing these uses; it also looks like test coverage may be spotty for some (StatsPrinter json/yaml part has no tests for example).

It is probably best to target this for "trunk" (4.1?).

 

Issue Links

• links to: GitHub Pull Request #1147
• links to: GitHub Pull Request #2297