[jira] [Commented] (KAFKA-2769) Integration tests for multi-consumer assignment including session timeouts

2015-11-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/KAFKA-2769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14997764#comment-14997764
 ] 

ASF GitHub Bot commented on KAFKA-2769:
---

Github user asfgit closed the pull request at:

https://github.com/apache/kafka/pull/472


> Integration tests for multi-consumer assignment including session timeouts
> --
>
> Key: KAFKA-2769
> URL: https://issues.apache.org/jira/browse/KAFKA-2769
> Project: Kafka
>  Issue Type: Test
>Reporter: Anna Povzner
>Assignee: Anna Povzner
> Fix For: 0.9.0.0
>
>
> We currently don't have integration tests for multi-consumer range assignment 
> and session timeout expiration scenarios.
> This card is to add multi-consumer integration tests for range assignment:
> -- initial assignment 
> -- adding a new consumer to an existing group
> -- adding and removing a topic to/from subscription
> -- session timeout expiration when consumer stops polling and on consumer 
> close().



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (KAFKA-2769) Integration tests for multi-consumer assignment including session timeouts

2015-11-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/KAFKA-2769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14997453#comment-14997453
 ] 

ASF GitHub Bot commented on KAFKA-2769:
---

GitHub user apovzner opened a pull request:

https://github.com/apache/kafka/pull/472

KAFKA-2769:  Multi-consumer integration tests for consumer assignment incl. 
session timeouts and corresponding fixes

-- Refactored multi-consumer integration group assignment validation tests 
for round-robin assignment
-- Added multi-consumer integration tests for session timeout expiration:
   1. When a consumer stops polling
2. When a consumer calls close()
-- Fixes to issues found with session timeout expiration tests woth help 
from Jason Gustafson: Try to avoid  SendFailedException exception by cancelling 
the scheduled tasks and ensuring metadata update before sending group leave 
requests + send leave group request with retries.

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

$ git pull https://github.com/apovzner/kafka cpkafka-81

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

https://github.com/apache/kafka/pull/472.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 #472


commit 0dc57aa7bed768559da19707e672dfbb55c0460b
Author: Anna Povzner 
Date:   2015-11-06T04:02:55Z

KAFKA-2769: Multi-consumer integration tests for consumer group subsribe 
and timeouts and consumer timeout fixes.

-- Refactored multi-consumer integration group assignment validation tests 
for round-robin assignment
-- Added multi-consumer integration tests for session timeout expiration
-- Fixes to issues found with session timeout expiration tests woth help 
from Jason Gustafson.
1. shouldKeepMemberAlive(): when we are in the sync phase, we do want 
to expire members if we don't get any response
2. Try to avoid  SendFailedException exception by cancelling the 
scheduled tasks and ensuring metadata update before sending group leave 
requests.

commit 87554675c0ad5e51602c4cc37a40a9bb273b6dd0
Author: Anna Povzner 
Date:   2015-11-07T00:10:04Z

KAFKA-2769: More reliable sending of leave group request in the consumer on 
consumer close().

commit 353ef1c079f39ad15a91ba26f64f95c541d517f1
Author: Anna Povzner 
Date:   2015-11-09T19:56:36Z

Reverted a change in shouldKeepMemberAlive() to check for sync callback set

commit 83f703d6cb8720ac12e25692ce273e15813e89f6
Author: Anna Povzner 
Date:   2015-11-09T20:52:15Z

fixed minor test issues




> Integration tests for multi-consumer assignment including session timeouts
> --
>
> Key: KAFKA-2769
> URL: https://issues.apache.org/jira/browse/KAFKA-2769
> Project: Kafka
>  Issue Type: Test
>Reporter: Anna Povzner
>Assignee: Anna Povzner
>
> We currently don't have integration tests for multi-consumer range assignment 
> and session timeout expiration scenarios.
> This card is to add multi-consumer integration tests for range assignment:
> -- initial assignment 
> -- adding a new consumer to an existing group
> -- adding and removing a topic to/from subscription
> -- session timeout expiration when consumer stops polling and on consumer 
> close().



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)