You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@samza.apache.org by Alan Lavintman <al...@gmail.com> on 2013/10/23 08:54:10 UTC

Take SAMZA-38

Hey guys, i have some free hours right now and Jay guided me to get into
Samza. For time differences (Israel) I would probably have some delays.

i took SAMZA-38 <https://issues.apache.org/jira/browse/SAMZA-38> and
modified the following files:

./docs/learn/documentation/0.7.0/jobs/configuration-table.html
./docs/learn/documentation/0.7.0/jobs/yarn-jobs.md
./docs/learn/documentation/0.7.0/jobs/job-runner.md
./docs/learn/documentation/0.7.0/jobs/packaging.md
./samza-core/src/main/scala/org/apache/samza/config/ShellCommandConfig.scala

Hows the review process?

Bests and thanks,
Alan.

Re: Take SAMZA-38

Posted by Chris Riccomini <cr...@linkedin.com>.
Hey Alan,

This is great! For instructions on contributing the patch, have a look at:

  http://samza.incubator.apache.org/contribute/rules.html

For SAMZA-38, the two main things are:

1. Upload your patch to the JIRA.
2. Open an RB ticket with your patch, and link to it from the JIRA.

Once that's done, we'll do a review and comment on the JIRA accordingly.

For reference, Samza's RB is located at:

  https://reviews.apache.org/groups/samza/

Let us know if you need any help.

Cheers,
Chris

On 10/22/13 11:54 PM, "Alan Lavintman" <al...@gmail.com> wrote:

>Hey guys, i have some free hours right now and Jay guided me to get into
>Samza. For time differences (Israel) I would probably have some delays.
>
>i took SAMZA-38 <https://issues.apache.org/jira/browse/SAMZA-38> and
>modified the following files:
>
>./docs/learn/documentation/0.7.0/jobs/configuration-table.html
>./docs/learn/documentation/0.7.0/jobs/yarn-jobs.md
>./docs/learn/documentation/0.7.0/jobs/job-runner.md
>./docs/learn/documentation/0.7.0/jobs/packaging.md
>./samza-core/src/main/scala/org/apache/samza/config/ShellCommandConfig.sca
>la
>
>Hows the review process?
>
>Bests and thanks,
>Alan.