You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by johnmshields <gi...@git.apache.org> on 2014/09/26 04:10:16 UTC

[GitHub] camel pull request: CAMEL-7864: Updates to allow setting Kafka's z...

GitHub user johnmshields opened a pull request:

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

    CAMEL-7864: Updates to allow setting Kafka's zookeeper.connect directly.

    - Added new property to KafkaConfiguration for zookeeperConnection and
      configured it to override the zookeeperHost and zookeeperPort
      properties.
    - Created getZookeeperConnect method on KafkaConfiguration to return
      the zookeeperConnect property if set or the combination of host ":"
      port if zookeeperConnect is not set.
    - Added zookeeperConnect get and set methods on KafkaEndpoint to
      delegate to KafkaConfiguration.
    - Updated KafkaConsumer to use the getZookeeperConnect method on the
      KafkaEndpoint.
    - Added tests for the changes.

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

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

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

    https://github.com/apache/camel/pull/281.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 #281
    
----
commit 64696f05293e55ec42f88ef2fed4e28cc9d52a5a
Author: john.shields <jo...@tubemogul.com>
Date:   2014-09-26T01:57:23Z

    CAMEL-7864: Updates to allow setting Kafka's zookeeper.connect directly.
    
    - Added new property to KafkaConfiguration for zookeeperConnection and
      configured it to override the zookeeperHost and zookeeperPort
      properties.
    - Created getZookeeperConnect method on KafkaConfiguration to return
      the zookeeperConnect property if set or the combination of host ":"
      port if zookeeperConnect is not set.
    - Added zookeeperConnect get and set methods on KafkaEndpoint to
      delegate to KafkaConfiguration.
    - Updated KafkaConsumer to use the getZookeeperConnect method on the
      KafkaEndpoint.
    - Added tests for the changes.

----


---
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] camel pull request: CAMEL-7864: Updates to allow setting Kafka's z...

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

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


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