Hi Jaikiran, do you mind utilizing the kafka patch review tool please https://cwiki.apache.org/confluence/display/KAFKA/Kafka+patch+review+tool as it is helpful for folks to comment on your code lines as it posts to review board for you and updates JIRA too.
/******************************************* Joe Stein Founder, Principal Consultant Big Data Open Source Security LLC http://www.stealth.ly Twitter: @allthingshadoop <http://www.twitter.com/allthingshadoop> ********************************************/ On Tue, Jan 6, 2015 at 11:26 PM, Jaikiran Pai <jai.forums2...@gmail.com> wrote: > Hello Kafka team, > > I was just looking around some newbie bugs in the JIRA, during the > weekend, and decided to take up https://issues.apache.org/ > jira/browse/KAFKA-1836. I've a patch, which I've attached to that JIRA a > few days back, which I would like to be reviewed. The patch handles the > case where the awaitUpdate timeout value for Metadata updates is lesser > than or equal to 0. It also has the documentation and config for > metadata.fetch.timeout.ms property updated. Finally it has a testcase > which verifies this change. > > Could one of you please take a look? > > > -Jaikiran >