[ 
https://issues.apache.org/jira/browse/CASSANDRA-16855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17401388#comment-17401388
 ] 

Tatu Saloranta commented on CASSANDRA-16855:
--------------------------------------------

One benchmark that seems quite relevant :

[https://github.com/fabienrenaud/java-json-benchmark]

It includes Jackson 2.10 as well as jsonsimple (in some places mentioned as 
"simplejson") – latter for just first 2 cases as it only supports non-POJO 
reading/writing (scalars, Lists, Maps) and not POJOs.
Benchmark is based on jmh and seems quite solid; it is written by someone other 
than author of either of json libraries (may be author of dsl-json/jsoniter, 
not 100% sure; but someone familiar with the domain). :)

Results suggest, fwtw, Jackson to be 2-3x faster at reading and 5x faster at 
writing of the stuff tested.

 

> Replace minor use of `json-simple` with Jackson
> -----------------------------------------------
>
>                 Key: CASSANDRA-16855
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16855
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Dependencies, Local/Other, Tool/nodetool
>            Reporter: Tatu Saloranta
>            Assignee: Tatu Saloranta
>            Priority: Normal
>              Labels: pull-request-available
>             Fix For: 4.x
>
>
> 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?).
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to