[infinispan-dev] optimistic locking - the time has come

Sanne Grinovero sanne at infinispan.org
Wed Sep 28 06:31:10 EDT 2011


Agreed, even better if we could use the commit description so it stays
with the code, and you can describe a step at a time.

We usually use a single-liner description, but for the more complex
patches we should add a paragraph. I did it occasionally, not sure
anybody noticed :)

Sanne

On 28 September 2011 11:27, Manik Surtani <manik at jboss.org> wrote:
>
> On 28 Sep 2011, at 10:05, Galder Zamarreño wrote:
>
> If you're gonna submit a big pull req, please use the description to
> *summarise the changes* in order to help reviewers. This helps focus the
> attention of the reviewer to the pieces of code that are most relevant.
>
> +1!
> --
> Manik Surtani
> manik at jboss.org
> twitter.com/maniksurtani
> Lead, Infinispan
> http://www.infinispan.org
>
>
>
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev
>



More information about the infinispan-dev mailing list