You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by edoardocomar <gi...@git.apache.org> on 2016/05/20 12:35:30 UTC

[GitHub] kafka pull request: KAFKA-3728 EndToEndAuthorizationTest offsets_t...

GitHub user edoardocomar opened a pull request:

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

    KAFKA-3728 EndToEndAuthorizationTest offsets_topic misconfigured

    Set OffsetsTopicReplicationFactorProp to 3 like MinInSyncReplicasProp
    and OffsetCommitRequiredAcksProp to 1 to avoid timeouts
    
    unit test for consumer that subscribes added 

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

    $ git pull https://github.com/edoardocomar/kafka KAFKA-3728

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

    https://github.com/apache/kafka/pull/1414.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 #1414
    
----
commit eab6d3f7aa1facc873332228fa32f35e98c02b2d
Author: Edoardo Comar <ec...@uk.ibm.com>
Date:   2016-05-19T15:52:15Z

    KAFKA-3728 EndToEndAuthorizationTest offsets_topic misconfigured
    
    Set OffsetsTopicReplicationFactorProp to 3 like MinInSyncReplicasProp
    and OffsetCommitRequiredAcksProp to 1 to avoid timeouts

----


---
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] kafka pull request: KAFKA-3728 EndToEndAuthorizationTest offsets_t...

Posted by edoardocomar <gi...@git.apache.org>.

Github user edoardocomar closed the pull request at:

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


---
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] kafka pull request: KAFKA-3728 EndToEndAuthorizationTest offsets_t...

Posted by edoardocomar <gi...@git.apache.org>.

GitHub user edoardocomar reopened a pull request:

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

    KAFKA-3728 EndToEndAuthorizationTest offsets_topic misconfigured

    Set OffsetsTopicReplicationFactorProp to 3 like MinInSyncReplicasProp
    and OffsetCommitRequiredAcksProp to 1 to avoid timeouts
    
    unit test for consumer that subscribes added 

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

    $ git pull https://github.com/edoardocomar/kafka KAFKA-3728

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

    https://github.com/apache/kafka/pull/1414.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 #1414
    
----
commit 82a1d1bb0ce3065d7008f18f9d1bcd84bcb24d94
Author: Edoardo Comar <ec...@uk.ibm.com>
Date:   2016-05-19T15:52:15Z

    KAFKA-3728 EndToEndAuthorizationTest offsets_topic misconfigured
    
    Set OffsetsTopicReplicationFactorProp to 3 like MinInSyncReplicasProp
    Earlier set acl for admin to read all
    OffsetCommitRequiredAcksProp to 1 
    
    Rationales:
    Moving the set acl for admin avoided errors like :
    
    ERROR [ReplicaFetcherThread-0-0], Error for partition
    [__consumer_offsets,0] to broker
    0:org.apache.kafka.common.errors.TopicAuthorizationException: Not
    authorized to access topics: [Topic authorization failed.]
    (kafka.server.ReplicaFetcherThread:97)
    
    that were happening before the read to all was set
    
    OffsetCommitRequiredAcksProp to 1 should not be needed, but without it,
    testProduceConsumeViaSubscribe fails if it's not the first one to be
    executed. Which suggests a cleanup problem, the test passes on its own.

----


---
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] kafka pull request: KAFKA-3728 EndToEndAuthorizationTest offsets_t...

Posted by edoardocomar <gi...@git.apache.org>.

Github user edoardocomar closed the pull request at:

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


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