You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by Neha Narkhede <ne...@gmail.com> on 2014/09/15 05:01:02 UTC
Re: Review Request 25044: KAFKA-1611 - Improve system test
configuration
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25044/#review53301
-----------------------------------------------------------
Ship it!
Ship It!
- Neha Narkhede
On Aug. 26, 2014, 12:04 a.m., Gwen Shapira wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25044/
> -----------------------------------------------------------
>
> (Updated Aug. 26, 2014, 12:04 a.m.)
>
>
> Review request for kafka.
>
>
> Repository: kafka
>
>
> Description
> -------
>
> make the config a bit more "out of the box" for the common case of local cluster. This includes:
> 1. Fix cluster_config.json of migration testsuite, it has hardcoded path that prevents it from working out of the box at all.
> 2. Use JAVA_HOME environment variable if "default" is specified and if JAVA_HOME is defined. The current guessing method is a bit broken and using JAVA_HOME will allow devs to configure their default java dir without editing multiple cluster_config.json files in multiple places.
>
>
> Diffs
> -----
>
> system_test/migration_tool_testsuite/cluster_config.json 8353e56
> system_test/utils/system_test_utils.py 50340f0
>
> Diff: https://reviews.apache.org/r/25044/diff/
>
>
> Testing
> -------
>
> Running system tests bunch of times :)
>
>
> Thanks,
>
> Gwen Shapira
>
>