Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@couchdb.apache.org
Received-SPF: pass (athena.apache.org: domain of randall.leeds@gmail.com
 designates 209.85.161.52 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type:content-transfer-encoding;
        b=x4VuDAhDnY3OyufgAcz+Rll3wySlPNm5RUvdq0m4j87uCRPHbPX/ceUW9bs7tw3SHP
         0AVzWVKpnYabGG72QIzYo/cniz3AH3CErHtx36W332bmTFPvkmgAdtHAqIffj2cM5uWl
         l0lmypFJhfYY9C5ImOb+h6UNf8DcoIp3btJlM=
MIME-Version: 1.0
In-Reply-To: <AANLkTikE3PC5iJV269tDA+-dMFhAGCQOk99xONmYuVyX@mail.gmail.com>
References: <AANLkTikf+tQN9sNBnsfWZcOUOG=1Vm7X83eGYT8f+135@mail.gmail.com>
	<AANLkTikQ_x5WnrdHnP5C9G81cPy8daGDZrz5KvyE2W0D@mail.gmail.com>
	<AANLkTim1UiYxJb9Qtx0RemqMN1N56iePAyBnVFfb5wTC@mail.gmail.com>
	<AANLkTikemAu=V9LC++Fj6jxDrgmS87Z+hvz_MZNdEVpf@mail.gmail.com>
	<AANLkTini9zHQEkt84XNfFCSpXBTEu8gpPLygmAauo_wj@mail.gmail.com>
	<AANLkTimcJD+n8RGqQ9fPO2ULJsu_WuEiHo_zr_ynL24q@mail.gmail.com>
	<AANLkTikE3PC5iJV269tDA+-dMFhAGCQOk99xONmYuVyX@mail.gmail.com>
Date: Thu, 14 Oct 2010 16:17:24 -0700
Message-ID: <AANLkTinm1Y3vse3que49goFgb3UrMiMcAn8kE2HBZt42@mail.gmail.com>
Subject: Re: filter changes for some docids
From: Randall Leeds <randall.leeds@gmail.com>
To: dev@couchdb.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

I don't mean to nag (much), but rather than let the patches get out of
date after you make more changes, would you mind looking at
https://issues.apache.org/jira/browse/COUCHDB-863
and
https://issues.apache.org/jira/browse/COUCHDB-704

They're both small bug fixes to the replication code and the first one
does some code cleanup on shutdown as well.
If there's any issue with them I'll fix it up.

On Thu, Oct 14, 2010 at 16:15, Randall Leeds <randall.leeds@gmail.com> wrot=
e:
> +1
>
> On Thu, Oct 14, 2010 at 16:05, Filipe David Manana <fdmanana@apache.org> =
wrote:
>> On Fri, Oct 15, 2010 at 12:04 AM, Filipe David Manana
>> <fdmanana@apache.org> wrote:
>>> Now that we have this, I think I should change the replicator to
>>> profit of this _changes feature and allow for continuous replication
>>> by doc IDs.
>>> Anyone against?
>>
>> Not only would allow for the continuous replication but also would
>> imply removing ugly code from the replicator
>> My +1
>>
>>>
>>>
>>> On Wed, Oct 13, 2010 at 10:10 PM, Benoit Chesneau <bchesneau@gmail.com>=
 wrote:
>>>> On Wed, Oct 13, 2010 at 6:21 PM, Paul Davis <paul.joseph.davis@gmail.c=
om> wrote:
>>>>> On Wed, Oct 13, 2010 at 12:05 PM, Filipe David Manana
>>>>> <fdmanana@apache.org> wrote:
>>>>>> For a matter of consistency with the replicator, I would name the
>>>>>> field as "doc_ids" instead of "docids".
>>>>>>
>>>>>> All the rest seems fine to me.
>>>>>>
>>>>>> On Wed, Oct 13, 2010 at 4:24 PM, Benoit Chesneau <bchesneau@gmail.co=
m> wrote:
>>>>>>> Hi all,
>>>>>>>
>>>>>>> For a project I have to know each time a design doc change and for =
now
>>>>>>> I have to create a filter function for each databases created to do
>>>>>>> that. I think getting changes on specific docids is a common behavi=
or
>>>>>>> (see replication for example) so I thought to add the possibility t=
o
>>>>>>> filters changes on docids if a list of documents is passed in the
>>>>>>> request.
>>>>>>>
>>>>>>> ex:
>>>>>>>
>>>>>>> POST /db/_changes
>>>>>>> {"docids": ["id1", ..]}
>>>>>>>
>>>>>>>
>>>>>>> Here is the patch that allows that feature :
>>>>>>> http://github.com/benoitc/couchdb/commit/fd7829b6b3093ab324a2b5c496=
600cc6335480b8
>>>>>>>
>>>>>>> What do you think about it ?
>>>>>>>
>>>>>>> - benoit
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Filipe David Manana,
>>>>>> fdmanana@gmail.com, fdmanana@apache.org
>>>>>>
>>>>>> "Reasonable men adapt themselves to the world.
>>>>>> =C2=A0Unreasonable men adapt the world to themselves.
>>>>>> =C2=A0That's why all progress depends on unreasonable men."
>>>>>>
>>>>>
>>>>> Looks fine to me. And I'd agree with filipe about using doc_ids for c=
onsistency.
>>>>>
>>>> commited with changes from docids to doc_ids + fixed safari test.
>>>> Thank for the review :)
>>>>
>>>> - benoit
>>>>
>>>
>>>
>>>
>>> --
>>> Filipe David Manana,
>>> fdmanana@gmail.com, fdmanana@apache.org
>>>
>>> "Reasonable men adapt themselves to the world.
>>> =C2=A0Unreasonable men adapt the world to themselves.
>>> =C2=A0That's why all progress depends on unreasonable men."
>>>
>>
>>
>>
>> --
>> Filipe David Manana,
>> fdmanana@gmail.com, fdmanana@apache.org
>>
>> "Reasonable men adapt themselves to the world.
>> =C2=A0Unreasonable men adapt the world to themselves.
>> =C2=A0That's why all progress depends on unreasonable men."
>>
>