GitHub user bostko opened a pull request:
https://github.com/apache/incubator-brooklyn/pull/742
Fix KafkaIntegrationTest tests
- updated kafka version
- installing from binary instead of compiling from source
Note on `KafkaIntegrationTest.testTwoBrokerCluster`:
The `getMessage()` return value is hardcoded since the new kafka client is
still [not
implemented](https://github.com/apache/kafka/blob/0.8.2/clients/src/main/java/org/apache/kafka/clients/consumer/KafkaConsumer.java#L553).
I tried other version kafka_2.10, 0.8.2-beta etc, but it is unfortunately
it is still not supported, it is available only in trunk.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bostko/incubator-brooklyn
fix_messaging_integration_tests
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-brooklyn/pull/742.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 #742
----
commit e7e330df9e22979ef622ddf66298fe6f54c0b9e9
Author: Valentin Aitken <[email protected]>
Date: 2015-06-26T10:13:21Z
Fix java version check
- if it is used openjdk java -version returns `openjdk version "1.8.0_45"'
which wasn't matched correctly previously
commit f3866bcbf6cb6959ef7f1cf9f0273df92524ec0c
Author: Valentin Aitken <[email protected]>
Date: 2015-07-06T15:41:23Z
Fix Kafka installation
- updated kafka version
- installing from binary instead of compiling from source
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
