Mailing-List: contact cassandra-user-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cassandra-user@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of jbellis@gmail.com designates
 74.125.78.147 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:from:date:message-id:subject:to
         :content-type:content-transfer-encoding;
        b=g6L6VTeTgcBnKjUBu0ccKyhkdtvzDC7/3INXA2hhuvOeGRjX4J1FU0wh3Whs3rbsz8
         30xyTDfjR950g8TAy2svZaQU45igtNv4V7pBH0ZfxfRn3JdgyCqv9uBQisPDrGRlPHyW
         bg2lq5/tI8z24c5h9noRchk5cZt0NGC+CctBU=
MIME-Version: 1.0
In-Reply-To: <e06563881002162107o3e2935d2w66fabb7fe24d1b0@mail.gmail.com>
References: <ad2266761002161711t5eb94515y7574acc3e4118a7@mail.gmail.com>
	<e06563881002161717o6bbeecd2u56f905eb2d86e547@mail.gmail.com>
	<e06563881002161720j3deb85b5v2a0e222a6e4f5bd9@mail.gmail.com>
	<ad2266761002161837p5f67ca4ag7b6140046c7e114b@mail.gmail.com>
	<e06563881002162107o3e2935d2w66fabb7fe24d1b0@mail.gmail.com>
From: Jonathan Ellis <jbellis@gmail.com>
Date: Tue, 16 Feb 2010 23:22:11 -0600
Message-ID: <e06563881002162122t4a99543g79adb4499050b084@mail.gmail.com>
Subject: Re: Testing row cache feature in trunk: write should put record in
	cache
To: cassandra-user@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

... tell you what, if you write the option-processing part in
DatabaseDescriptor I will do the actual cache part. :)

On Tue, Feb 16, 2010 at 11:07 PM, Jonathan Ellis <jbellis@gmail.com> wrote:
> https://issues.apache.org/jira/secure/CreateIssue!default.jspa, but
> this is pretty low priority for me.
>
> On Tue, Feb 16, 2010 at 8:37 PM, Weijun Li <weijunli@gmail.com> wrote:
>> Just tried to make quick change to enable it but it didn't work out :-(
>>
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 ColumnFamily cachedRow =3D cf=
s.getRawCachedRow(mutation.key());
>>
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 // What I modified
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 if( cachedRow =3D=3D null =
) {
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 =A0=A0=A0 cfs.cacheRow(mut=
ation.key());
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 =A0=A0=A0 cachedRow =3D cf=
s.getRawCachedRow(mutation.key());
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 }
>>
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 if (cachedRow !=3D null)
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 =A0=A0=A0 cachedRow.addAll=
(columnFamily);
>>
>> How can I open a ticket for you to make the change (enable row cache wri=
te
>> through with an option)?
>>
>> Thanks,
>> -Weijun
>>
>> On Tue, Feb 16, 2010 at 5:20 PM, Jonathan Ellis <jbellis@gmail.com> wrot=
e:
>>>
>>> On Tue, Feb 16, 2010 at 7:17 PM, Jonathan Ellis <jbellis@gmail.com> wro=
te:
>>> > On Tue, Feb 16, 2010 at 7:11 PM, Weijun Li <weijunli@gmail.com> wrote=
:
>>> >> Just started to play with the row cache feature in trunk: it seems t=
o
>>> >> be
>>> >> working fine so far except that for RowsCached parameter you need to
>>> >> specify
>>> >> number of rows rather than a percentage (e.g., "20%" doesn't work).
>>> >
>>> > 20% works, but it's 20% of the rows at server startup. =A0So on a fre=
sh
>>> > start that is zero.
>>> >
>>> > Maybe we should just get rid of the % feature...
>>>
>>> (Actually, it shouldn't be hard to update this on flush, if you want
>>> to open a ticket.)
>>
>>
>