Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
From: williamd1618 <git@git.apache.org>
To: dev@camel.apache.org
Reply-To: dev@camel.apache.org
Message-ID: <git-pr-596-camel@git.apache.org>
Subject: [GitHub] camel pull request: Fix Endpoint URI for KafkProducer 
Content-Type: text/plain
Date: Fri, 21 Aug 2015 20:30:55 +0000 (UTC)

GitHub user williamd1618 opened a pull request:

    https://github.com/apache/camel/pull/596

    Fix Endpoint URI for KafkProducer 

    When doing a route from Kafka topic to Kafka topic I noticed that functionality like:
    
    ```
    from("kafka:localhost:9092?topic=from&zookeeperHost=localhost&groupId=mytest&zookeeperPort=2181&autoOffsetReset=smallest).to("kafka:localhost:9092?topic=to&zookeeperHost=localhost&groupId=mytest&zookeeperPort=2181);
    ```
    
    did not work as the Exchange header of kafka.TOPIC was being used instead of the UriParam in the KafkaEndpoint for the producer.  

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

    $ git pull https://github.com/williamd1618/camel master

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

    https://github.com/apache/camel/pull/596.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 #596
    
----
commit 7ed60926e3e617f9f47f07db6e6ae55667a8db03
Author: Daniel Williams <daniel.williams@pearson.com>
Date:   2015-08-21T20:27:50Z

    Fix Endpoint URI for KafkProduer to guarantee it reads topic UriParam it it exists on KafkaEndpoint

----


---
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.
---