Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KAFKA-4461 Added support to ProcessorTopologyTestDriver for internal topics. #2499

Conversation

amccague
Copy link
Contributor

@amccague amccague commented Feb 4, 2017

This resolves an issue in driving tests using the ProcessorTopologyTestDriver when groupBy() is invoked downstream of a processor that flags repartitioning.

Ticket: https://issues.apache.org/jira/browse/KAFKA-4461
Discussion: http://search-hadoop.com/m/Kafka/uyzND1wbKeY1Q8nH1

@dguy @guozhangwang

The contribution is my original work and I license the work to the project under the project's open source license.

Copy link
Contributor

@dguy dguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @amccague - LGTM

Copy link
Contributor

@guozhangwang guozhangwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and merged to trunk.

@asfgit asfgit closed this in 1f8a2ad Feb 6, 2017
soenkeliebau pushed a commit to soenkeliebau/kafka that referenced this pull request Feb 7, 2017
… topics

This resolves an issue in driving tests using the ProcessorTopologyTestDriver when `groupBy()` is invoked downstream of a processor that flags repartitioning.

Ticket: https://issues.apache.org/jira/browse/KAFKA-4461
Discussion: http://search-hadoop.com/m/Kafka/uyzND1wbKeY1Q8nH1

dguy guozhangwang

The contribution is my original work and I license the work to the project under the project's open source license.

Author: Adrian McCague <amccague@gmail.com>

Reviewers: Damian Guy, Guozhang Wang

Closes apache#2499 from amccague/KAFKA-4461_ProcessorTopologyTestDriver_map_groupbykey
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants