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

Andrii Biletskyi commented on KAFKA-2229:
-----------------------------------------

Sorry I was on vacation so didn't have time to work on this issue. I think I 
resolved all your comments now I need to rebase changes to the trunk and commit 
corresponding fixes per your comments. It's been a while since last rebase, but 
still I think it will be quicker if I do it.
Your help will be very appreciated in reviewing again after rebase. As I can 
see there were some changes in replica assignment code which my changes use a 
lot, this part is pretty critical. Also you could provide more details (or even 
recommendations) to your comment 
"This logic is fairly complex and difficult to track. I am not sure the best 
way to resolve it. Breaking it into smaller functions may help."
-> I've updated on it in RB.

Thanks, 
Andrii

> Phase 1: Requests and KafkaApis
> -------------------------------
>
>                 Key: KAFKA-2229
>                 URL: https://issues.apache.org/jira/browse/KAFKA-2229
>             Project: Kafka
>          Issue Type: Sub-task
>            Reporter: Andrii Biletskyi
>            Assignee: Andrii Biletskyi
>         Attachments: KAFKA-2229.patch, KAFKA-2229_2015-06-30_16:59:17.patch
>
>




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

Reply via email to