Review Board 1.7.22


Patch for KAFKA-1134

Review Request #15953 - Created Dec. 3, 2013 and updated

Guozhang Wang
KAFKA-1134
Reviewers
kafka
kafka
KAFKA-1134.v1

 
Total:
4
Open:
3
Resolved:
1
Dropped:
0
Status:
From:
Review request changed
Updated (Dec. 5, 2013, 7:13 p.m.)
Posted (Dec. 9, 2013, 9:58 p.m.)

   

  
Is this change supposed to be part of this patch?
is this intended? This is required no?
why is this required?
Posted (Dec. 9, 2013, 11:01 p.m.)

   

  
This it not required, but just to eliminate log flushing due to fire-all-watcher-at-zk-expiration.
Neha, I think you reviewed the wrong diff, it is diff between r1 and r2.
Ship it!
Posted (Dec. 10, 2013, 11:48 p.m.)
Ship It!