+1.The "different color for newest patch" doesn't work very well if you are
color blind, so I do appreciate a revision number in the name.
From: Yongjun Zhang <yzh...@cloudera.com>
To: common-dev@hadoop.apache.org
Sent: Tuesday, November 25, 2014 11:37 PM
Subject: Re: a friendly suggestion for developers when uploading patches
Thanks Harsh for the info and Andrew for sharing the script. It looks that
the script is intelligent enough to pick the latest attachment even if all
attachments have the same name.
Yet, I hope we use the following as the guideline for patch names:
<*projectName*>-<*jiraNum*>-<*revNum*>.patch
So we can easily identify individual patch revs.
Thanks.
--Yongjun
On Tue, Nov 25, 2014 at 5:54 PM, Andrew Wang <andrew.w...@cloudera.com>
wrote:
> This might be a good time to mention my fetch-patch script, I use it to
> easily download the latest attachment on a jira:
>
> https://github.com/umbrant/dotfiles/blob/master/bin/fetch-patch
>
> On Tue, Nov 25, 2014 at 5:44 PM, Harsh J <ha...@cloudera.com> wrote:
>
> > For the same filename, you can observe also that the JIRA colors the
> > latest one to be different than the older ones automatically - this is
> > what I rely on.
> >
> > On Sat, Nov 22, 2014 at 12:36 AM, Yongjun Zhang <yzh...@cloudera.com>
> > wrote:
> > > Hi,
> > >
> > > When I look at patches uploaded to jiras, from time to time I notice
> that
> > > different revisions of the patch is uploaded with the same patch file
> > name,
> > > some time for quite a few times. It's confusing which is which.
> > >
> > > I'd suggest that as a guideline, we do the following when uploading a
> > patch:
> > >
> > > - include a revision number in the patch file name.A
> > > - include a comment, stating that a new patch is uploaded, including
> > the
> > > revision number of the patch in the comment.
> > >
> > > This way, it's easier to refer to a specific version of a patch, and to
> > > know which patch a comment is made about.
> > >
> > > Hope that makes sense to you.
> > >
> > > Thanks.
> > >
> > > --Yongjun
> >
> >
> >
> > --
> > Harsh J
> >
>
