The change looks good. Would you mind attaching a patch to the jira? This will indicate that you are ok with donating the code to Apache.
Thanks, Jun On Wed, Jan 30, 2013 at 1:21 PM, fsaintjacques <g...@git.apache.org> wrote: > GitHub user fsaintjacques opened a pull request: > > https://github.com/apache/kafka/pull/2 > > KAFKA-294 > > This issue can be caused by a non-existing path but also a > misunderstanding from the config file. A short example will help the user. > > You can merge this pull request into a Git repository by running: > > $ git pull https://github.com/fsaintjacques/kafka trunk > > Alternatively you can review and apply these changes as the patch at: > > https://github.com/apache/kafka/pull/2.patch > > ---- > commit e6ea059efb1365d9e186f4883bb6747a9425f365 > Author: François Saint-Jacques <fsaintjacq...@gmail.com> > Date: 2013-01-30T21:17:19Z > > zk.connect clarification in server.properties > > The previous wording is not 100% clear, a little example clarifies it. > > ---- > >
