You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@couchdb.apache.org by rn...@apache.org on 2010/08/19 10:06:24 UTC

svn commit: r987065 - /couchdb/branches/improve_range_support/

Author: rnewson
Date: Thu Aug 19 08:06:23 2010
New Revision: 987065

URL: http://svn.apache.org/viewvc?rev=987065&view=rev
Log:
Create branch improve_range_support

Added:
    couchdb/branches/improve_range_support/   (props changed)
      - copied from r986634, couchdb/trunk/

Propchange: couchdb/branches/improve_range_support/
------------------------------------------------------------------------------
--- svn:ignore (added)
+++ svn:ignore Thu Aug 19 08:06:23 2010
@@ -0,0 +1,23 @@
+INSTALL
+ChangeLog
+LICENCE
+Makefile
+Makefile.in
+acinclude.m4
+aclocal.m4
+apache-couchdb-*
+autom4te.cache
+build-aux
+config.h
+config.h.in
+config.log
+config.status
+configure
+couchdb-*.tar.gz
+couchdb-*.zip
+cover
+libtool
+stamp-h1
+*.gz
+tmp
+erl_crash.dump

Propchange: couchdb/branches/improve_range_support/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Thu Aug 19 08:06:23 2010
@@ -0,0 +1,8 @@
+/couchdb/branches/0.10.x:820161,823015
+/couchdb/branches/0.9.x:775634
+/couchdb/branches/account:896157-896984
+/couchdb/branches/design_resources:751716-751803
+/couchdb/branches/form:729440-730015
+/couchdb/branches/list-iterator:782292-784593
+/couchdb/branches/tail_header:775760-778477
+/couchdb/tags/0.10.0:825400

Re: svn commit: r987065 - /couchdb/branches/improve_range_support/

Posted by J Chris Anderson <jc...@apache.org>.

On Aug 19, 2010, at 3:54 AM, Noah Slater wrote:

> 
> On 19 Aug 2010, at 09:58, Robert Newson wrote:
> 
>> It seems impossible to guess when I should put things in trunk or on
>> branches. :)
> 
> I would use the following rule of thumb:
> 
> If the change is self-contained and relatively minor, commit to trunk.
> 

I think it's fine to commit large patches directly to trunk, as long as you are reasonably sure they work and won't surprise people.

> Otherwise, a branch.
> 
> Basically, avoid breaking trunk between commits.

Re: svn commit: r987065 - /couchdb/branches/improve_range_support/

Posted by Noah Slater <ns...@apache.org>.

On 19 Aug 2010, at 09:58, Robert Newson wrote:

> It seems impossible to guess when I should put things in trunk or on
> branches. :)

I would use the following rule of thumb:

If the change is self-contained and relatively minor, commit to trunk.

Otherwise, a branch.

Basically, avoid breaking trunk between commits.

Re: svn commit: r987065 - /couchdb/branches/improve_range_support/

Posted by Robert Newson <ro...@gmail.com>.

It seems impossible to guess when I should put things in trunk or on
branches. :)

I've put the work on trunk now. Attachments honor the Range header in
all cases now. Attachments written after my code change, or old
attachments after a compaction, are served more efficiently.

B.

On Thu, Aug 19, 2010 at 9:13 AM, Benoit Chesneau <bc...@gmail.com> wrote:
> Why  branch now ? If something need to be fixed or improved vs the
> current status in trunk I would prefer it done as quickly as possible
> in trunk .
>
> Shouldn't we revert the commit in trunk too in other cases ?
>
> - benoit
>
> On Thu, Aug 19, 2010 at 10:06 AM,  <rn...@apache.org> wrote:
>> Author: rnewson
>> Date: Thu Aug 19 08:06:23 2010
>> New Revision: 987065
>>
>> URL: http://svn.apache.org/viewvc?rev=987065&view=rev
>> Log:
>> Create branch improve_range_support
>>
>> Added:
>>    couchdb/branches/improve_range_support/   (props changed)
>>      - copied from r986634, couchdb/trunk/
>>
>> Propchange: couchdb/branches/improve_range_support/
>> ------------------------------------------------------------------------------
>> --- svn:ignore (added)
>> +++ svn:ignore Thu Aug 19 08:06:23 2010
>> @@ -0,0 +1,23 @@
>> +INSTALL
>> +ChangeLog
>> +LICENCE
>> +Makefile
>> +Makefile.in
>> +acinclude.m4
>> +aclocal.m4
>> +apache-couchdb-*
>> +autom4te.cache
>> +build-aux
>> +config.h
>> +config.h.in
>> +config.log
>> +config.status
>> +configure
>> +couchdb-*.tar.gz
>> +couchdb-*.zip
>> +cover
>> +libtool
>> +stamp-h1
>> +*.gz
>> +tmp
>> +erl_crash.dump
>>
>> Propchange: couchdb/branches/improve_range_support/
>> ------------------------------------------------------------------------------
>> --- svn:mergeinfo (added)
>> +++ svn:mergeinfo Thu Aug 19 08:06:23 2010
>> @@ -0,0 +1,8 @@
>> +/couchdb/branches/0.10.x:820161,823015
>> +/couchdb/branches/0.9.x:775634
>> +/couchdb/branches/account:896157-896984
>> +/couchdb/branches/design_resources:751716-751803
>> +/couchdb/branches/form:729440-730015
>> +/couchdb/branches/list-iterator:782292-784593
>> +/couchdb/branches/tail_header:775760-778477
>> +/couchdb/tags/0.10.0:825400
>>
>>
>>
>

Re: svn commit: r987065 - /couchdb/branches/improve_range_support/

Posted by Benoit Chesneau <bc...@gmail.com>.

Why  branch now ? If something need to be fixed or improved vs the
current status in trunk I would prefer it done as quickly as possible
in trunk .

Shouldn't we revert the commit in trunk too in other cases ?

- benoit

On Thu, Aug 19, 2010 at 10:06 AM,  <rn...@apache.org> wrote:
> Author: rnewson
> Date: Thu Aug 19 08:06:23 2010
> New Revision: 987065
>
> URL: http://svn.apache.org/viewvc?rev=987065&view=rev
> Log:
> Create branch improve_range_support
>
> Added:
>    couchdb/branches/improve_range_support/   (props changed)
>      - copied from r986634, couchdb/trunk/
>
> Propchange: couchdb/branches/improve_range_support/
> ------------------------------------------------------------------------------
> --- svn:ignore (added)
> +++ svn:ignore Thu Aug 19 08:06:23 2010
> @@ -0,0 +1,23 @@
> +INSTALL
> +ChangeLog
> +LICENCE
> +Makefile
> +Makefile.in
> +acinclude.m4
> +aclocal.m4
> +apache-couchdb-*
> +autom4te.cache
> +build-aux
> +config.h
> +config.h.in
> +config.log
> +config.status
> +configure
> +couchdb-*.tar.gz
> +couchdb-*.zip
> +cover
> +libtool
> +stamp-h1
> +*.gz
> +tmp
> +erl_crash.dump
>
> Propchange: couchdb/branches/improve_range_support/
> ------------------------------------------------------------------------------
> --- svn:mergeinfo (added)
> +++ svn:mergeinfo Thu Aug 19 08:06:23 2010
> @@ -0,0 +1,8 @@
> +/couchdb/branches/0.10.x:820161,823015
> +/couchdb/branches/0.9.x:775634
> +/couchdb/branches/account:896157-896984
> +/couchdb/branches/design_resources:751716-751803
> +/couchdb/branches/form:729440-730015
> +/couchdb/branches/list-iterator:782292-784593
> +/couchdb/branches/tail_header:775760-778477
> +/couchdb/tags/0.10.0:825400
>
>
>