I've now updated the patch submission page to include instructions for installation on Debian based systems and also added an overview section to help understand what the Kafka patch submission tool is for https://cwiki.apache.org/confluence/display/KAFKA/Patch+submission+and+review

-Jaikiran

On Wednesday 07 January 2015 10:41 AM, Jaikiran Pai wrote:
Thanks Joe, I'll do that. I had indeed quickly looked at that document over the weekend but I saw that it was missing instructions for Debian based systems (am on LinuxMint) and so decided to skip that for now till I understand the whole tool chain. I'll see which of these tools I need to install and how to set them up (and if needed add those instructions to that document)

-Jaikiran
On Wednesday 07 January 2015 10:04 AM, Joe Stein wrote:
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



Reply via email to