GitHub user ijuma opened a pull request:

    https://github.com/apache/kafka/pull/55

    Kafka 1595 remove deprecated json parser

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ijuma/kafka 
kafka-1595-remove-deprecated-json-parser

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/55.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #55
    
----
commit 31ac16dbd9d657fbb9a8a63153361ec543aabb7b
Author: Ismael Juma <ism...@juma.me.uk>
Date:   2015-04-17T23:26:28Z

    Drop support for Scala 2.9.1 and 2.9.2
    
    Use the same `scalatest` version for all Scala
    versions and remove unused code.

commit ab71458018dc2c5b05a981da1a73aea08a0163cd
Author: Ismael Juma <ism...@juma.me.uk>
Date:   2015-04-20T20:53:54Z

    Introduce `testJsonParse`
    
    Simple test that shows existing behaviour.

commit bda759a01cf244d0786830a7e9540b2e84a620c2
Author: Ismael Juma <ism...@juma.me.uk>
Date:   2015-04-21T00:15:02Z

    KAFKA-1595; Remove deprecated and slower scala JSON parser from 
kafka.consumer.TopicCount
    
    A combination of spray-json's AST combined with jawn's parser are used as 
the replacement.
    Note that both libraries have no dependencies and are relatively simple. We 
use `jawn` for
    its performance, but it could be dropped by changing one line in 
`Json.parseFull`.
    
    An attempt has been made to maintain the existing behaviour regarding when 
exceptions
    are thrown. There are a number of cases where `DeserializationException` 
will be thrown
    instead of `ClassCastException`, however. It is expected that users would 
not try to catch
    `ClassCastException`.

commit d1adee404e6d9a7f7b07da4d7ab2ed8e2087ccdf
Author: Ismael Juma <ism...@juma.me.uk>
Date:   2015-04-21T00:23:39Z

    Minor clean-ups in `Json.encode`

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to