You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@couchdb.apache.org by Randall Leeds <ra...@gmail.com> on 2012/10/11 01:24:41 UTC

1.2.x Hotfixes

Hey devs,

Before 1.2 shipped, I made some hotfixes on the 1.2.x branch (I know,
I'm sorry).

I finally went through them. I posted a branch called 'hotfixes1.2' to
the asf repo.

Here are all the changes that I had made, along with the commit on the
hotfix branch that corresponds.



These are related to improving logging during file operation failures:

    04c4a1041de06e8e39983cf9616c8baf9d889106
    ede9482fc3c9e629572ab1376d88f4499c4c8beb
    cd238b42d1333cece5ab899c35c3b83ada8d448a
    af9e1c883ca4accb209cccc7ac9d26efa4daf1fd
    ba8fa86890087650a02a8ebaaea2537468999472

    ... it is encompassed by 6e3f17a85b3de5d3c5ff2865bc58a1b9a6a47fab


This one logs all 500 errors at the httpd level:

    6887953432e9ccdef4bb8785b782fc53f6e61602

    ... it is exactly 205ccb07114107e6daf7a156764c751e2f9f1571



This one updated NEWS & CHANGES for the above:

    25e079117218b52c18d51dca0810b974ec80ff8a

    ... And I took the liberty of including the below as well
    and put all the notice in a1cc138689973f5f6e5ff68aa9204dbe1ddb179c


This one fixed COUCHDB-1445:

    94e72e7fcc66f6740a3ec517a1bccf2cfa024cf7

    ... is exactly f79591d73c7fa02f2938875bceac691703938f81 except I
    left out the test because it was strange to begin with.


This one is no longer necessary, as the new 1.3 indexer code never had this bug:

    a38f59d1752ec611f0edc26e8f641a36b52f0c67


Okay to merge? Thanks for looking.

Re: 1.2.x Hotfixes

Posted by Randall Leeds <ra...@gmail.com>.
Reviewed by rnewson and committed.

On Thu, Oct 11, 2012 at 2:42 AM, Noah Slater <ns...@tumbolia.org> wrote:
> Thanks. +1 on getting these sorted, natch
>
> On Thu, Oct 11, 2012 at 10:37 AM, Robert Newson <ro...@gmail.com>wrote:
>
>> I can review on Saturday if no one gets to it.
>>
>> Noah, that's how I read it, these are fixes in 1.2 that aren't on
>> master, so they'd be regressions without Randall's merge.
>>
>> Sent from the ocean floor
>>
>> On 11 Oct 2012, at 10:28, Noah Slater <ns...@tumbolia.org> wrote:
>>
>> > These were fixes that landed on the 1.2.x branch, but were never ported
>> to
>> > master?
>> >
>> > I assume that's what you're proposing to do now?
>> >
>> > On Thu, Oct 11, 2012 at 12:24 AM, Randall Leeds <randall.leeds@gmail.com
>> >wrote:
>> >
>> >> Hey devs,
>> >>
>> >> Before 1.2 shipped, I made some hotfixes on the 1.2.x branch (I know,
>> >> I'm sorry).
>> >>
>> >> I finally went through them. I posted a branch called 'hotfixes1.2' to
>> >> the asf repo.
>> >>
>> >> Here are all the changes that I had made, along with the commit on the
>> >> hotfix branch that corresponds.
>> >>
>> >>
>> >>
>> >> These are related to improving logging during file operation failures:
>> >>
>> >>    04c4a1041de06e8e39983cf9616c8baf9d889106
>> >>    ede9482fc3c9e629572ab1376d88f4499c4c8beb
>> >>    cd238b42d1333cece5ab899c35c3b83ada8d448a
>> >>    af9e1c883ca4accb209cccc7ac9d26efa4daf1fd
>> >>    ba8fa86890087650a02a8ebaaea2537468999472
>> >>
>> >>    ... it is encompassed by 6e3f17a85b3de5d3c5ff2865bc58a1b9a6a47fab
>> >>
>> >>
>> >> This one logs all 500 errors at the httpd level:
>> >>
>> >>    6887953432e9ccdef4bb8785b782fc53f6e61602
>> >>
>> >>    ... it is exactly 205ccb07114107e6daf7a156764c751e2f9f1571
>> >>
>> >>
>> >>
>> >> This one updated NEWS & CHANGES for the above:
>> >>
>> >>    25e079117218b52c18d51dca0810b974ec80ff8a
>> >>
>> >>    ... And I took the liberty of including the below as well
>> >>    and put all the notice in a1cc138689973f5f6e5ff68aa9204dbe1ddb179c
>> >>
>> >>
>> >> This one fixed COUCHDB-1445:
>> >>
>> >>    94e72e7fcc66f6740a3ec517a1bccf2cfa024cf7
>> >>
>> >>    ... is exactly f79591d73c7fa02f2938875bceac691703938f81 except I
>> >>    left out the test because it was strange to begin with.
>> >>
>> >>
>> >> This one is no longer necessary, as the new 1.3 indexer code never had
>> >> this bug:
>> >>
>> >>    a38f59d1752ec611f0edc26e8f641a36b52f0c67
>> >>
>> >>
>> >> Okay to merge? Thanks for looking.
>> >>
>> >
>> >
>> >
>> > --
>> > NS
>>
>
>
>
> --
> NS

Re: 1.2.x Hotfixes

Posted by Noah Slater <ns...@tumbolia.org>.
Thanks. +1 on getting these sorted, natch

On Thu, Oct 11, 2012 at 10:37 AM, Robert Newson <ro...@gmail.com>wrote:

> I can review on Saturday if no one gets to it.
>
> Noah, that's how I read it, these are fixes in 1.2 that aren't on
> master, so they'd be regressions without Randall's merge.
>
> Sent from the ocean floor
>
> On 11 Oct 2012, at 10:28, Noah Slater <ns...@tumbolia.org> wrote:
>
> > These were fixes that landed on the 1.2.x branch, but were never ported
> to
> > master?
> >
> > I assume that's what you're proposing to do now?
> >
> > On Thu, Oct 11, 2012 at 12:24 AM, Randall Leeds <randall.leeds@gmail.com
> >wrote:
> >
> >> Hey devs,
> >>
> >> Before 1.2 shipped, I made some hotfixes on the 1.2.x branch (I know,
> >> I'm sorry).
> >>
> >> I finally went through them. I posted a branch called 'hotfixes1.2' to
> >> the asf repo.
> >>
> >> Here are all the changes that I had made, along with the commit on the
> >> hotfix branch that corresponds.
> >>
> >>
> >>
> >> These are related to improving logging during file operation failures:
> >>
> >>    04c4a1041de06e8e39983cf9616c8baf9d889106
> >>    ede9482fc3c9e629572ab1376d88f4499c4c8beb
> >>    cd238b42d1333cece5ab899c35c3b83ada8d448a
> >>    af9e1c883ca4accb209cccc7ac9d26efa4daf1fd
> >>    ba8fa86890087650a02a8ebaaea2537468999472
> >>
> >>    ... it is encompassed by 6e3f17a85b3de5d3c5ff2865bc58a1b9a6a47fab
> >>
> >>
> >> This one logs all 500 errors at the httpd level:
> >>
> >>    6887953432e9ccdef4bb8785b782fc53f6e61602
> >>
> >>    ... it is exactly 205ccb07114107e6daf7a156764c751e2f9f1571
> >>
> >>
> >>
> >> This one updated NEWS & CHANGES for the above:
> >>
> >>    25e079117218b52c18d51dca0810b974ec80ff8a
> >>
> >>    ... And I took the liberty of including the below as well
> >>    and put all the notice in a1cc138689973f5f6e5ff68aa9204dbe1ddb179c
> >>
> >>
> >> This one fixed COUCHDB-1445:
> >>
> >>    94e72e7fcc66f6740a3ec517a1bccf2cfa024cf7
> >>
> >>    ... is exactly f79591d73c7fa02f2938875bceac691703938f81 except I
> >>    left out the test because it was strange to begin with.
> >>
> >>
> >> This one is no longer necessary, as the new 1.3 indexer code never had
> >> this bug:
> >>
> >>    a38f59d1752ec611f0edc26e8f641a36b52f0c67
> >>
> >>
> >> Okay to merge? Thanks for looking.
> >>
> >
> >
> >
> > --
> > NS
>



-- 
NS

Re: 1.2.x Hotfixes

Posted by Robert Newson <ro...@gmail.com>.
I can review on Saturday if no one gets to it.

Noah, that's how I read it, these are fixes in 1.2 that aren't on
master, so they'd be regressions without Randall's merge.

Sent from the ocean floor

On 11 Oct 2012, at 10:28, Noah Slater <ns...@tumbolia.org> wrote:

> These were fixes that landed on the 1.2.x branch, but were never ported to
> master?
>
> I assume that's what you're proposing to do now?
>
> On Thu, Oct 11, 2012 at 12:24 AM, Randall Leeds <ra...@gmail.com>wrote:
>
>> Hey devs,
>>
>> Before 1.2 shipped, I made some hotfixes on the 1.2.x branch (I know,
>> I'm sorry).
>>
>> I finally went through them. I posted a branch called 'hotfixes1.2' to
>> the asf repo.
>>
>> Here are all the changes that I had made, along with the commit on the
>> hotfix branch that corresponds.
>>
>>
>>
>> These are related to improving logging during file operation failures:
>>
>>    04c4a1041de06e8e39983cf9616c8baf9d889106
>>    ede9482fc3c9e629572ab1376d88f4499c4c8beb
>>    cd238b42d1333cece5ab899c35c3b83ada8d448a
>>    af9e1c883ca4accb209cccc7ac9d26efa4daf1fd
>>    ba8fa86890087650a02a8ebaaea2537468999472
>>
>>    ... it is encompassed by 6e3f17a85b3de5d3c5ff2865bc58a1b9a6a47fab
>>
>>
>> This one logs all 500 errors at the httpd level:
>>
>>    6887953432e9ccdef4bb8785b782fc53f6e61602
>>
>>    ... it is exactly 205ccb07114107e6daf7a156764c751e2f9f1571
>>
>>
>>
>> This one updated NEWS & CHANGES for the above:
>>
>>    25e079117218b52c18d51dca0810b974ec80ff8a
>>
>>    ... And I took the liberty of including the below as well
>>    and put all the notice in a1cc138689973f5f6e5ff68aa9204dbe1ddb179c
>>
>>
>> This one fixed COUCHDB-1445:
>>
>>    94e72e7fcc66f6740a3ec517a1bccf2cfa024cf7
>>
>>    ... is exactly f79591d73c7fa02f2938875bceac691703938f81 except I
>>    left out the test because it was strange to begin with.
>>
>>
>> This one is no longer necessary, as the new 1.3 indexer code never had
>> this bug:
>>
>>    a38f59d1752ec611f0edc26e8f641a36b52f0c67
>>
>>
>> Okay to merge? Thanks for looking.
>>
>
>
>
> --
> NS

Re: 1.2.x Hotfixes

Posted by Noah Slater <ns...@tumbolia.org>.
These were fixes that landed on the 1.2.x branch, but were never ported to
master?

I assume that's what you're proposing to do now?

On Thu, Oct 11, 2012 at 12:24 AM, Randall Leeds <ra...@gmail.com>wrote:

> Hey devs,
>
> Before 1.2 shipped, I made some hotfixes on the 1.2.x branch (I know,
> I'm sorry).
>
> I finally went through them. I posted a branch called 'hotfixes1.2' to
> the asf repo.
>
> Here are all the changes that I had made, along with the commit on the
> hotfix branch that corresponds.
>
>
>
> These are related to improving logging during file operation failures:
>
>     04c4a1041de06e8e39983cf9616c8baf9d889106
>     ede9482fc3c9e629572ab1376d88f4499c4c8beb
>     cd238b42d1333cece5ab899c35c3b83ada8d448a
>     af9e1c883ca4accb209cccc7ac9d26efa4daf1fd
>     ba8fa86890087650a02a8ebaaea2537468999472
>
>     ... it is encompassed by 6e3f17a85b3de5d3c5ff2865bc58a1b9a6a47fab
>
>
> This one logs all 500 errors at the httpd level:
>
>     6887953432e9ccdef4bb8785b782fc53f6e61602
>
>     ... it is exactly 205ccb07114107e6daf7a156764c751e2f9f1571
>
>
>
> This one updated NEWS & CHANGES for the above:
>
>     25e079117218b52c18d51dca0810b974ec80ff8a
>
>     ... And I took the liberty of including the below as well
>     and put all the notice in a1cc138689973f5f6e5ff68aa9204dbe1ddb179c
>
>
> This one fixed COUCHDB-1445:
>
>     94e72e7fcc66f6740a3ec517a1bccf2cfa024cf7
>
>     ... is exactly f79591d73c7fa02f2938875bceac691703938f81 except I
>     left out the test because it was strange to begin with.
>
>
> This one is no longer necessary, as the new 1.3 indexer code never had
> this bug:
>
>     a38f59d1752ec611f0edc26e8f641a36b52f0c67
>
>
> Okay to merge? Thanks for looking.
>



-- 
NS