You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by KyleWinkelman <gi...@git.apache.org> on 2017/05/03 02:00:32 UTC

[GitHub] kafka pull request #2963: Minor: Removed use of keySerde in CachingSessionSt...

GitHub user KyleWinkelman opened a pull request:

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

    Minor: Removed use of keySerde in CachingSessionStore.

    CachingSessionStore wasn't properly using the default keySerde if no Serde was supplied. I saw the below error in the logs for one of my test cases.
    
    ERROR stream-thread [cogroup-integration-test-3-5570fe48-d2a3-4271-80b1-81962295553d-StreamThread-6] Streams application error during processing:  (org.apache.kafka.streams.processor.internals.StreamThread:335)
    java.lang.NullPointerException
    	at org.apache.kafka.streams.state.internals.CachingSessionStore.findSessions(CachingSessionStore.java:93)
    	at org.apache.kafka.streams.kstream.internals.KStreamSessionWindowAggregate$KStreamSessionWindowAggregateProcessor.process(KStreamSessionWindowAggregate.java:94)
    	at org.apache.kafka.streams.processor.internals.ProcessorNode$1.run(ProcessorNode.java:47)
    	at org.apache.kafka.streams.processor.internals.StreamsMetricsImpl.measureLatencyNs(StreamsMetricsImpl.java:187)
    	at org.apache.kafka.streams.processor.internals.ProcessorNode.process(ProcessorNode.java:133)
    	at org.apache.kafka.streams.processor.internals.ProcessorContextImpl.forward(ProcessorContextImpl.java:82)
    	at org.apache.kafka.streams.processor.internals.SourceNode.process(SourceNode.java:69)
    	at org.apache.kafka.streams.processor.internals.StreamTask.process(StreamTask.java:206)
    	at org.apache.kafka.streams.processor.internals.StreamThread.processAndPunctuate(StreamThread.java:657)
    	at org.apache.kafka.streams.processor.internals.StreamThread.runLoop(StreamThread.java:728)
    	at org.apache.kafka.streams.processor.internals.StreamThread.run(StreamThread.java:327)

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

    $ git pull https://github.com/KyleWinkelman/kafka CachingSessionStore-fix-keySerde-use

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

    https://github.com/apache/kafka/pull/2963.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 #2963
    
----
commit 41592448a401434df475a1dfb2ab2c4d2b6a0508
Author: Kyle Winkelman <ky...@optum.com>
Date:   2017-05-03T01:52:05Z

    Removed use of keySerde in CachingSessionStore and substituted serdes
    which will contain the default keySerde if one is not supplied.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] kafka pull request #2963: Kafka-5205: Removed use of keySerde in CachingSess...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---