camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jay mann (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (CAMEL-10087) camel-kafka does not work in OSGI container
Date Tue, 28 Jun 2016 16:12:57 GMT

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

Jay mann edited comment on CAMEL-10087 at 6/28/16 4:12 PM:
-----------------------------------------------------------

I see you implemented the fix.  But you did not add the deserializers for the consumer.  You
need to add these keys in your updateClassProperties method:

key.deserializer
value.deserializer

btw, you should probably use the kafka constants as well:

ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG
ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG
ProducerConfig.KEY_SERIALIZER_CLASS_CONFIG
ProducerConfig.VALUE_SERIALIZER_CLASS_CONFIG

I can open another PR if you want me to.

-J



was (Author: jmandawg):
I see you implemented the fix.  But you did not add the deserializers for the consumer.  You
need to add these keys in your updateClassProperties method:

key.deserializer
value.deserializer


-J


> camel-kafka does not work in OSGI container
> -------------------------------------------
>
>                 Key: CAMEL-10087
>                 URL: https://issues.apache.org/jira/browse/CAMEL-10087
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-kafka
>    Affects Versions: 2.17.1
>            Reporter: Jay mann
>
> Currently this component is broken in OSGI environment due to this:
> https://issues.apache.org/jira/browse/KAFKA-3218
> We can leave it broken until they get their bugs worked out.  Or i can submit a PR with
an intermediate fix that should work, as described in the jira noted above.
> https://github.com/apache/camel/pull/1049



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message