You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Lefty Leverenz <le...@gmail.com> on 2015/05/01 00:53:11 UTC

Re: Please double-check JIRA number on commit

Found a negative answer to #2 in some ASF instructions
<http://www.apache.org/security/committers.html> for security issues
(*underline
added*):

15.  The log for the svn commit that applied the fix is updated to include
> the CVE number. Projects that use git as their primary source code control
> system should not do this as *editing a pushed commit causes all sorts of
> problems*.


All the more reason to be careful about the JIRA number when you commit a
patch.


-- Lefty

On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <le...@gmail.com>
wrote:

> Since switching to git, three commits have had the wrong JIRA number.  Two
> were typos and the third one gave the umbrella issue's number.  Fortunately
> the correct issue could be found by searching for the summary text.
>
>    1. How can we reduce this failure rate?
>    2. Can JIRA numbers be fixed after-the-fact in commit records?
>    3. Should JIRA issues track commit IDs?
>
> -- Lefty
>

Re: Please double-check JIRA number on commit

Posted by Lefty Leverenz <le...@gmail.com>.
Perhaps we need a comments column for issues that aren't
wrong-JIRA-numbers.  For example, HIVE-11536
<https://issues.apache.org/jira/browse/HIVE-11536> which just got added to
the list (commit b22e54ecec96d1b86fea1b53dae4c8bb68a11f9c).

The errata.txt file says "Commits with the wrong JIRA referenced" but it
could say "Commits with the wrong JIRA referenced (or other error as
noted)".

-- Lefty


On Mon, Aug 31, 2015 at 7:35 PM, Owen O'Malley <om...@apache.org> wrote:

> Ok, I've created a patch with Lefty's items. Please take a look:
>
> https://issues.apache.org/jira/browse/HIVE-11704
>

Re: Please double-check JIRA number on commit

Posted by Owen O'Malley <om...@apache.org>.
Ok, I've created a patch with Lefty's items. Please take a look:

https://issues.apache.org/jira/browse/HIVE-11704

Re: Please double-check JIRA number on commit

Posted by Chao Sun <ch...@cloudera.com>.
Sorry, I forgot to include a JIRA number for HIVE-10021. Thanks Lefty for
finding out.
I wish there's a way to fix this.

Best,
Chao

On Sat, Aug 29, 2015 at 10:45 PM, Lefty Leverenz <le...@gmail.com>
wrote:

> One more for the list of commits that don't have the right JIRA issue
> number -- but in this case, it doesn't have any issue number:
>
>    - dcf21cd6fa98fb5db01ef661bb3b9f94d9ca2d15  HIVE-10021
>    <https://issues.apache.org/jira/browse/HIVE-10021>
>
> Do we want to put the list into the source tree?
>
> -- Lefty
>
>
> On Sun, Jun 7, 2015 at 10:32 PM, Lefty Leverenz <le...@gmail.com>
> wrote:
>
> > Here are the ones I know about:
> >
> >    - 5a576b6fbf1680ab4dd8f275cad484a2614ef2c1  HIVE-10391
> >    <https://issues.apache.org/jira/browse/HIVE-10391>
> >    - 582f4e1bc39b9605d11f762480b29561a44688ae  HIVE-10217
> >    <https://issues.apache.org/jira/browse/HIVE-10217>
> >    - 8981f365bf0cf921bc0ac2ff8914df44ca2f7de7  HIVE-10500
> >    <https://issues.apache.org/jira/browse/HIVE-10500>
> >    - 09100831adff7589ee48e735a4beac6ebb25cb3e
> >    & f3ab5fda6af57afff31c29ad048d906fd095d5fb  HIVE-10885
> >    <https://issues.apache.org/jira/browse/HIVE-10885>
> >
> > Those have JIRA comments listing the commit IDs.  Should we list commit
> > IDs on all issues?
> >
> > -- Lefty
> >
> > On Sun, Jun 7, 2015 at 2:16 PM, Lefty Leverenz <le...@gmail.com>
> > wrote:
> >
> >> Good idea!  I'll compile a list.
> >>
> >> -- Lefty
> >>
> >> On Sat, Jun 6, 2015 at 9:00 AM, Owen O'Malley <om...@apache.org>
> wrote:
> >>
> >>> Editing commits creates a lot of problems. May I suggest that we start
> a
> >>> file that lists the exceptions that you've noticed and check that into
> >>> our
> >>> source tree? That will give everyone access to fix things when they
> >>> recognize it. It would look like:
> >>>
> >>> 75b679c54c9088434678da2a4d89122620903d72 hive-1234
> >>> e8520424879d07e50ea07c28a671251e95c382f6 hive-1235
> >>>
> >>> Of course there may be multiple commits for the same jira, especially
> on
> >>> different branches.
> >>>
> >>> .. Owen
> >>>
> >>> .. Owen
> >>>
> >>> On Sat, Jun 6, 2015 at 2:30 AM, Lefty Leverenz <
> leftyleverenz@gmail.com>
> >>> wrote:
> >>>
> >>> > >
> >>> > > ... we should put the commit log in the corresponding
> >>> > > JIRA comment as a cross-reference.
> >>> > >
> >>> >
> >>> > I've only noticed one more slip-up since the end of April, but still
> >>> think
> >>> > this is a good idea.
> >>> >
> >>> > Any more suggestions?
> >>> >
> >>> >
> >>> > -- Lefty
> >>> >
> >>> > On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang <xz...@cloudera.com>
> >>> wrote:
> >>> >
> >>> > > Yes, for that, I think we should put the commit log in the
> >>> corresponding
> >>> > > JIRA comment as a cross-reference. I have to admit that I had to
> fix
> >>> > commit
> >>> > > message for a few of my svn commits in the past.
> >>> > >
> >>> > > On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <
> >>> leftyleverenz@gmail.com
> >>> > >
> >>> > > wrote:
> >>> > >
> >>> > > > Found a negative answer to #2 in some ASF instructions
> >>> > > > <http://www.apache.org/security/committers.html> for security
> >>> issues
> >>> > > > (*underline
> >>> > > > added*):
> >>> > > >
> >>> > > > 15.  The log for the svn commit that applied the fix is updated
> to
> >>> > > include
> >>> > > > > the CVE number. Projects that use git as their primary source
> >>> code
> >>> > > > control
> >>> > > > > system should not do this as *editing a pushed commit causes
> all
> >>> > sorts
> >>> > > of
> >>> > > > > problems*.
> >>> > > >
> >>> > > >
> >>> > > > All the more reason to be careful about the JIRA number when you
> >>> > commit a
> >>> > > > patch.
> >>> > > >
> >>> > > >
> >>> > > > -- Lefty
> >>> > > >
> >>> > > > On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <
> >>> > leftyleverenz@gmail.com
> >>> > > >
> >>> > > > wrote:
> >>> > > >
> >>> > > > > Since switching to git, three commits have had the wrong JIRA
> >>> number.
> >>> > > > Two
> >>> > > > > were typos and the third one gave the umbrella issue's number.
> >>> > > > Fortunately
> >>> > > > > the correct issue could be found by searching for the summary
> >>> text.
> >>> > > > >
> >>> > > > >    1. How can we reduce this failure rate?
> >>> > > > >    2. Can JIRA numbers be fixed after-the-fact in commit
> records?
> >>> > > > >    3. Should JIRA issues track commit IDs?
> >>> > > > >
> >>> > > > > -- Lefty
> >>> > > > >
> >>> > > >
> >>> > >
> >>> >
> >>>
> >>
> >>
> >
>

Re: Please double-check JIRA number on commit

Posted by Alan Gates <al...@gmail.com>.
+1 to putting it in source.  We could start an "errata" file or 
something that lists these.

Alan.

> Lefty Leverenz <ma...@gmail.com>
> August 29, 2015 at 22:45
> One more for the list of commits that don't have the right JIRA issue
> number -- but in this case, it doesn't have any issue number:
>
> - dcf21cd6fa98fb5db01ef661bb3b9f94d9ca2d15 HIVE-10021
> <https://issues.apache.org/jira/browse/HIVE-10021>
>
> Do we want to put the list into the source tree?
>
> -- Lefty
>
>
> On Sun, Jun 7, 2015 at 10:32 PM, Lefty Leverenz <le...@gmail.com>
>
> Lefty Leverenz <ma...@gmail.com>
> June 7, 2015 at 19:32
> Here are the ones I know about:
>
> - 5a576b6fbf1680ab4dd8f275cad484a2614ef2c1 HIVE-10391
> <https://issues.apache.org/jira/browse/HIVE-10391>
> - 582f4e1bc39b9605d11f762480b29561a44688ae HIVE-10217
> <https://issues.apache.org/jira/browse/HIVE-10217>
> - 8981f365bf0cf921bc0ac2ff8914df44ca2f7de7 HIVE-10500
> <https://issues.apache.org/jira/browse/HIVE-10500>
> - 09100831adff7589ee48e735a4beac6ebb25cb3e
> & f3ab5fda6af57afff31c29ad048d906fd095d5fb HIVE-10885
> <https://issues.apache.org/jira/browse/HIVE-10885>
>
> Those have JIRA comments listing the commit IDs. Should we list commit IDs
> on all issues?
>
> -- Lefty
>
> On Sun, Jun 7, 2015 at 2:16 PM, Lefty Leverenz <le...@gmail.com>
>
> Lefty Leverenz <ma...@gmail.com>
> June 7, 2015 at 14:16
> Good idea! I'll compile a list.
>
> -- Lefty
>
>
> Owen O'Malley <ma...@apache.org>
> June 6, 2015 at 9:00
> Editing commits creates a lot of problems. May I suggest that we start a
> file that lists the exceptions that you've noticed and check that into our
> source tree? That will give everyone access to fix things when they
> recognize it. It would look like:
>
> 75b679c54c9088434678da2a4d89122620903d72 hive-1234
> e8520424879d07e50ea07c28a671251e95c382f6 hive-1235
>
> Of course there may be multiple commits for the same jira, especially on
> different branches.
>
> .. Owen
>
> .. Owen
>
> On Sat, Jun 6, 2015 at 2:30 AM, Lefty Leverenz <le...@gmail.com>
>
> Lefty Leverenz <ma...@gmail.com>
> June 6, 2015 at 2:30
>> ... we should put the commit log in the corresponding
>> JIRA comment as a cross-reference.
>>
>
> I've only noticed one more slip-up since the end of April, but still think
> this is a good idea.
>
> Any more suggestions?
>
>
> -- Lefty
>
> On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang<xz...@cloudera.com>  wrote:
>
>> Yes, for that, I think we should put the commit log in the corresponding
>> JIRA comment as a cross-reference. I have to admit that I had to fix commit
>> message for a few of my svn commits in the past.
>>
>> On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz<le...@gmail.com>
>> wrote:
>>
>>> Found a negative answer to #2 in some ASF instructions
>>> <http://www.apache.org/security/committers.html>  for security issues
>>> (*underline
>>> added*):
>>>
>>> 15.  The log for the svn commit that applied the fix is updated to
>> include
>>>> the CVE number. Projects that use git as their primary source code
>>> control
>>>> system should not do this as *editing a pushed commit causes all sorts
>> of
>>>> problems*.
>>> All the more reason to be careful about the JIRA number when you commit a
>>> patch.
>>>
>>>
>>> -- Lefty
>>>
>>> On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz<leftyleverenz@gmail.com
>>>
>>> wrote:
>>>
>>>> Since switching to git, three commits have had the wrong JIRA number.
>>> Two
>>>> were typos and the third one gave the umbrella issue's number.
>>> Fortunately
>>>> the correct issue could be found by searching for the summary text.
>>>>
>>>>     1. How can we reduce this failure rate?
>>>>     2. Can JIRA numbers be fixed after-the-fact in commit records?
>>>>     3. Should JIRA issues track commit IDs?
>>>>
>>>> -- Lefty
>>>>
>

Re: Please double-check JIRA number on commit

Posted by Lefty Leverenz <le...@gmail.com>.
One more for the list of commits that don't have the right JIRA issue
number -- but in this case, it doesn't have any issue number:

   - dcf21cd6fa98fb5db01ef661bb3b9f94d9ca2d15  HIVE-10021
   <https://issues.apache.org/jira/browse/HIVE-10021>

Do we want to put the list into the source tree?

-- Lefty


On Sun, Jun 7, 2015 at 10:32 PM, Lefty Leverenz <le...@gmail.com>
wrote:

> Here are the ones I know about:
>
>    - 5a576b6fbf1680ab4dd8f275cad484a2614ef2c1  HIVE-10391
>    <https://issues.apache.org/jira/browse/HIVE-10391>
>    - 582f4e1bc39b9605d11f762480b29561a44688ae  HIVE-10217
>    <https://issues.apache.org/jira/browse/HIVE-10217>
>    - 8981f365bf0cf921bc0ac2ff8914df44ca2f7de7  HIVE-10500
>    <https://issues.apache.org/jira/browse/HIVE-10500>
>    - 09100831adff7589ee48e735a4beac6ebb25cb3e
>    & f3ab5fda6af57afff31c29ad048d906fd095d5fb  HIVE-10885
>    <https://issues.apache.org/jira/browse/HIVE-10885>
>
> Those have JIRA comments listing the commit IDs.  Should we list commit
> IDs on all issues?
>
> -- Lefty
>
> On Sun, Jun 7, 2015 at 2:16 PM, Lefty Leverenz <le...@gmail.com>
> wrote:
>
>> Good idea!  I'll compile a list.
>>
>> -- Lefty
>>
>> On Sat, Jun 6, 2015 at 9:00 AM, Owen O'Malley <om...@apache.org> wrote:
>>
>>> Editing commits creates a lot of problems. May I suggest that we start a
>>> file that lists the exceptions that you've noticed and check that into
>>> our
>>> source tree? That will give everyone access to fix things when they
>>> recognize it. It would look like:
>>>
>>> 75b679c54c9088434678da2a4d89122620903d72 hive-1234
>>> e8520424879d07e50ea07c28a671251e95c382f6 hive-1235
>>>
>>> Of course there may be multiple commits for the same jira, especially on
>>> different branches.
>>>
>>> .. Owen
>>>
>>> .. Owen
>>>
>>> On Sat, Jun 6, 2015 at 2:30 AM, Lefty Leverenz <le...@gmail.com>
>>> wrote:
>>>
>>> > >
>>> > > ... we should put the commit log in the corresponding
>>> > > JIRA comment as a cross-reference.
>>> > >
>>> >
>>> > I've only noticed one more slip-up since the end of April, but still
>>> think
>>> > this is a good idea.
>>> >
>>> > Any more suggestions?
>>> >
>>> >
>>> > -- Lefty
>>> >
>>> > On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang <xz...@cloudera.com>
>>> wrote:
>>> >
>>> > > Yes, for that, I think we should put the commit log in the
>>> corresponding
>>> > > JIRA comment as a cross-reference. I have to admit that I had to fix
>>> > commit
>>> > > message for a few of my svn commits in the past.
>>> > >
>>> > > On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <
>>> leftyleverenz@gmail.com
>>> > >
>>> > > wrote:
>>> > >
>>> > > > Found a negative answer to #2 in some ASF instructions
>>> > > > <http://www.apache.org/security/committers.html> for security
>>> issues
>>> > > > (*underline
>>> > > > added*):
>>> > > >
>>> > > > 15.  The log for the svn commit that applied the fix is updated to
>>> > > include
>>> > > > > the CVE number. Projects that use git as their primary source
>>> code
>>> > > > control
>>> > > > > system should not do this as *editing a pushed commit causes all
>>> > sorts
>>> > > of
>>> > > > > problems*.
>>> > > >
>>> > > >
>>> > > > All the more reason to be careful about the JIRA number when you
>>> > commit a
>>> > > > patch.
>>> > > >
>>> > > >
>>> > > > -- Lefty
>>> > > >
>>> > > > On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <
>>> > leftyleverenz@gmail.com
>>> > > >
>>> > > > wrote:
>>> > > >
>>> > > > > Since switching to git, three commits have had the wrong JIRA
>>> number.
>>> > > > Two
>>> > > > > were typos and the third one gave the umbrella issue's number.
>>> > > > Fortunately
>>> > > > > the correct issue could be found by searching for the summary
>>> text.
>>> > > > >
>>> > > > >    1. How can we reduce this failure rate?
>>> > > > >    2. Can JIRA numbers be fixed after-the-fact in commit records?
>>> > > > >    3. Should JIRA issues track commit IDs?
>>> > > > >
>>> > > > > -- Lefty
>>> > > > >
>>> > > >
>>> > >
>>> >
>>>
>>
>>
>

Re: Please double-check JIRA number on commit

Posted by Lefty Leverenz <le...@gmail.com>.
Here are the ones I know about:

   - 5a576b6fbf1680ab4dd8f275cad484a2614ef2c1  HIVE-10391
   <https://issues.apache.org/jira/browse/HIVE-10391>
   - 582f4e1bc39b9605d11f762480b29561a44688ae  HIVE-10217
   <https://issues.apache.org/jira/browse/HIVE-10217>
   - 8981f365bf0cf921bc0ac2ff8914df44ca2f7de7  HIVE-10500
   <https://issues.apache.org/jira/browse/HIVE-10500>
   - 09100831adff7589ee48e735a4beac6ebb25cb3e
   & f3ab5fda6af57afff31c29ad048d906fd095d5fb  HIVE-10885
   <https://issues.apache.org/jira/browse/HIVE-10885>

Those have JIRA comments listing the commit IDs.  Should we list commit IDs
on all issues?

-- Lefty

On Sun, Jun 7, 2015 at 2:16 PM, Lefty Leverenz <le...@gmail.com>
wrote:

> Good idea!  I'll compile a list.
>
> -- Lefty
>
> On Sat, Jun 6, 2015 at 9:00 AM, Owen O'Malley <om...@apache.org> wrote:
>
>> Editing commits creates a lot of problems. May I suggest that we start a
>> file that lists the exceptions that you've noticed and check that into our
>> source tree? That will give everyone access to fix things when they
>> recognize it. It would look like:
>>
>> 75b679c54c9088434678da2a4d89122620903d72 hive-1234
>> e8520424879d07e50ea07c28a671251e95c382f6 hive-1235
>>
>> Of course there may be multiple commits for the same jira, especially on
>> different branches.
>>
>> .. Owen
>>
>> .. Owen
>>
>> On Sat, Jun 6, 2015 at 2:30 AM, Lefty Leverenz <le...@gmail.com>
>> wrote:
>>
>> > >
>> > > ... we should put the commit log in the corresponding
>> > > JIRA comment as a cross-reference.
>> > >
>> >
>> > I've only noticed one more slip-up since the end of April, but still
>> think
>> > this is a good idea.
>> >
>> > Any more suggestions?
>> >
>> >
>> > -- Lefty
>> >
>> > On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang <xz...@cloudera.com>
>> wrote:
>> >
>> > > Yes, for that, I think we should put the commit log in the
>> corresponding
>> > > JIRA comment as a cross-reference. I have to admit that I had to fix
>> > commit
>> > > message for a few of my svn commits in the past.
>> > >
>> > > On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <
>> leftyleverenz@gmail.com
>> > >
>> > > wrote:
>> > >
>> > > > Found a negative answer to #2 in some ASF instructions
>> > > > <http://www.apache.org/security/committers.html> for security
>> issues
>> > > > (*underline
>> > > > added*):
>> > > >
>> > > > 15.  The log for the svn commit that applied the fix is updated to
>> > > include
>> > > > > the CVE number. Projects that use git as their primary source code
>> > > > control
>> > > > > system should not do this as *editing a pushed commit causes all
>> > sorts
>> > > of
>> > > > > problems*.
>> > > >
>> > > >
>> > > > All the more reason to be careful about the JIRA number when you
>> > commit a
>> > > > patch.
>> > > >
>> > > >
>> > > > -- Lefty
>> > > >
>> > > > On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <
>> > leftyleverenz@gmail.com
>> > > >
>> > > > wrote:
>> > > >
>> > > > > Since switching to git, three commits have had the wrong JIRA
>> number.
>> > > > Two
>> > > > > were typos and the third one gave the umbrella issue's number.
>> > > > Fortunately
>> > > > > the correct issue could be found by searching for the summary
>> text.
>> > > > >
>> > > > >    1. How can we reduce this failure rate?
>> > > > >    2. Can JIRA numbers be fixed after-the-fact in commit records?
>> > > > >    3. Should JIRA issues track commit IDs?
>> > > > >
>> > > > > -- Lefty
>> > > > >
>> > > >
>> > >
>> >
>>
>
>

Re: Please double-check JIRA number on commit

Posted by Lefty Leverenz <le...@gmail.com>.
Good idea!  I'll compile a list.

-- Lefty

On Sat, Jun 6, 2015 at 9:00 AM, Owen O'Malley <om...@apache.org> wrote:

> Editing commits creates a lot of problems. May I suggest that we start a
> file that lists the exceptions that you've noticed and check that into our
> source tree? That will give everyone access to fix things when they
> recognize it. It would look like:
>
> 75b679c54c9088434678da2a4d89122620903d72 hive-1234
> e8520424879d07e50ea07c28a671251e95c382f6 hive-1235
>
> Of course there may be multiple commits for the same jira, especially on
> different branches.
>
> .. Owen
>
> .. Owen
>
> On Sat, Jun 6, 2015 at 2:30 AM, Lefty Leverenz <le...@gmail.com>
> wrote:
>
> > >
> > > ... we should put the commit log in the corresponding
> > > JIRA comment as a cross-reference.
> > >
> >
> > I've only noticed one more slip-up since the end of April, but still
> think
> > this is a good idea.
> >
> > Any more suggestions?
> >
> >
> > -- Lefty
> >
> > On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang <xz...@cloudera.com>
> wrote:
> >
> > > Yes, for that, I think we should put the commit log in the
> corresponding
> > > JIRA comment as a cross-reference. I have to admit that I had to fix
> > commit
> > > message for a few of my svn commits in the past.
> > >
> > > On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <
> leftyleverenz@gmail.com
> > >
> > > wrote:
> > >
> > > > Found a negative answer to #2 in some ASF instructions
> > > > <http://www.apache.org/security/committers.html> for security issues
> > > > (*underline
> > > > added*):
> > > >
> > > > 15.  The log for the svn commit that applied the fix is updated to
> > > include
> > > > > the CVE number. Projects that use git as their primary source code
> > > > control
> > > > > system should not do this as *editing a pushed commit causes all
> > sorts
> > > of
> > > > > problems*.
> > > >
> > > >
> > > > All the more reason to be careful about the JIRA number when you
> > commit a
> > > > patch.
> > > >
> > > >
> > > > -- Lefty
> > > >
> > > > On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <
> > leftyleverenz@gmail.com
> > > >
> > > > wrote:
> > > >
> > > > > Since switching to git, three commits have had the wrong JIRA
> number.
> > > > Two
> > > > > were typos and the third one gave the umbrella issue's number.
> > > > Fortunately
> > > > > the correct issue could be found by searching for the summary text.
> > > > >
> > > > >    1. How can we reduce this failure rate?
> > > > >    2. Can JIRA numbers be fixed after-the-fact in commit records?
> > > > >    3. Should JIRA issues track commit IDs?
> > > > >
> > > > > -- Lefty
> > > > >
> > > >
> > >
> >
>

Re: Please double-check JIRA number on commit

Posted by Owen O'Malley <om...@apache.org>.
Editing commits creates a lot of problems. May I suggest that we start a
file that lists the exceptions that you've noticed and check that into our
source tree? That will give everyone access to fix things when they
recognize it. It would look like:

75b679c54c9088434678da2a4d89122620903d72 hive-1234
e8520424879d07e50ea07c28a671251e95c382f6 hive-1235

Of course there may be multiple commits for the same jira, especially on
different branches.

.. Owen

.. Owen

On Sat, Jun 6, 2015 at 2:30 AM, Lefty Leverenz <le...@gmail.com>
wrote:

> >
> > ... we should put the commit log in the corresponding
> > JIRA comment as a cross-reference.
> >
>
> I've only noticed one more slip-up since the end of April, but still think
> this is a good idea.
>
> Any more suggestions?
>
>
> -- Lefty
>
> On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang <xz...@cloudera.com> wrote:
>
> > Yes, for that, I think we should put the commit log in the corresponding
> > JIRA comment as a cross-reference. I have to admit that I had to fix
> commit
> > message for a few of my svn commits in the past.
> >
> > On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <leftyleverenz@gmail.com
> >
> > wrote:
> >
> > > Found a negative answer to #2 in some ASF instructions
> > > <http://www.apache.org/security/committers.html> for security issues
> > > (*underline
> > > added*):
> > >
> > > 15.  The log for the svn commit that applied the fix is updated to
> > include
> > > > the CVE number. Projects that use git as their primary source code
> > > control
> > > > system should not do this as *editing a pushed commit causes all
> sorts
> > of
> > > > problems*.
> > >
> > >
> > > All the more reason to be careful about the JIRA number when you
> commit a
> > > patch.
> > >
> > >
> > > -- Lefty
> > >
> > > On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <
> leftyleverenz@gmail.com
> > >
> > > wrote:
> > >
> > > > Since switching to git, three commits have had the wrong JIRA number.
> > > Two
> > > > were typos and the third one gave the umbrella issue's number.
> > > Fortunately
> > > > the correct issue could be found by searching for the summary text.
> > > >
> > > >    1. How can we reduce this failure rate?
> > > >    2. Can JIRA numbers be fixed after-the-fact in commit records?
> > > >    3. Should JIRA issues track commit IDs?
> > > >
> > > > -- Lefty
> > > >
> > >
> >
>

Re: Please double-check JIRA number on commit

Posted by Lefty Leverenz <le...@gmail.com>.
>
> ... we should put the commit log in the corresponding
> JIRA comment as a cross-reference.
>

I've only noticed one more slip-up since the end of April, but still think
this is a good idea.

Any more suggestions?


-- Lefty

On Thu, Apr 30, 2015 at 4:53 PM, Xuefu Zhang <xz...@cloudera.com> wrote:

> Yes, for that, I think we should put the commit log in the corresponding
> JIRA comment as a cross-reference. I have to admit that I had to fix commit
> message for a few of my svn commits in the past.
>
> On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <le...@gmail.com>
> wrote:
>
> > Found a negative answer to #2 in some ASF instructions
> > <http://www.apache.org/security/committers.html> for security issues
> > (*underline
> > added*):
> >
> > 15.  The log for the svn commit that applied the fix is updated to
> include
> > > the CVE number. Projects that use git as their primary source code
> > control
> > > system should not do this as *editing a pushed commit causes all sorts
> of
> > > problems*.
> >
> >
> > All the more reason to be careful about the JIRA number when you commit a
> > patch.
> >
> >
> > -- Lefty
> >
> > On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <leftyleverenz@gmail.com
> >
> > wrote:
> >
> > > Since switching to git, three commits have had the wrong JIRA number.
> > Two
> > > were typos and the third one gave the umbrella issue's number.
> > Fortunately
> > > the correct issue could be found by searching for the summary text.
> > >
> > >    1. How can we reduce this failure rate?
> > >    2. Can JIRA numbers be fixed after-the-fact in commit records?
> > >    3. Should JIRA issues track commit IDs?
> > >
> > > -- Lefty
> > >
> >
>

Re: Please double-check JIRA number on commit

Posted by Xuefu Zhang <xz...@cloudera.com>.
Yes, for that, I think we should put the commit log in the corresponding
JIRA comment as a cross-reference. I have to admit that I had to fix commit
message for a few of my svn commits in the past.

On Thu, Apr 30, 2015 at 3:53 PM, Lefty Leverenz <le...@gmail.com>
wrote:

> Found a negative answer to #2 in some ASF instructions
> <http://www.apache.org/security/committers.html> for security issues
> (*underline
> added*):
>
> 15.  The log for the svn commit that applied the fix is updated to include
> > the CVE number. Projects that use git as their primary source code
> control
> > system should not do this as *editing a pushed commit causes all sorts of
> > problems*.
>
>
> All the more reason to be careful about the JIRA number when you commit a
> patch.
>
>
> -- Lefty
>
> On Thu, Apr 30, 2015 at 2:57 AM, Lefty Leverenz <le...@gmail.com>
> wrote:
>
> > Since switching to git, three commits have had the wrong JIRA number.
> Two
> > were typos and the third one gave the umbrella issue's number.
> Fortunately
> > the correct issue could be found by searching for the summary text.
> >
> >    1. How can we reduce this failure rate?
> >    2. Can JIRA numbers be fixed after-the-fact in commit records?
> >    3. Should JIRA issues track commit IDs?
> >
> > -- Lefty
> >
>