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

Posted to jira@kafka.apache.org by "Viktor Somogyi (JIRA)" <ji...@apache.org> on 2019/02/14 14:09:00 UTC

[jira] [Commented] (KAFKA-7736) Consolidate Map usages in TransactionManager

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

Viktor Somogyi commented on KAFKA-7736:
---------------------------------------

[~hachikuji] Had a go on refactoring these map usages into a single map. My approach was to create a TopicPartition to TopicPartitionEntry map which contains the values of the old maps, that way we can treat these data together. Please have a look once you have some time.

> Consolidate Map usages in TransactionManager
> --------------------------------------------
>
>                 Key: KAFKA-7736
>                 URL: https://issues.apache.org/jira/browse/KAFKA-7736
>             Project: Kafka
>          Issue Type: Task
>          Components: producer 
>            Reporter: Viktor Somogyi
>            Assignee: Viktor Somogyi
>            Priority: Minor
>              Labels: exactly-once
>
> There are a bunch of Map<TopicPartition, X> collections in TransactionManager which could be consolidated into a single map to consolidate bookkeeping and get rid of potential bugs.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)