Mailing-List: contact cassandra-user-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cassandra-user@incubator.apache.org
Received-SPF: pass (nike.apache.org: local policy)
From: Daniel Kluesing <dk@bluekai.com>
To: "cassandra-user@incubator.apache.org"
	<cassandra-user@incubator.apache.org>
Date: Mon, 8 Mar 2010 11:25:08 -0800
Subject: RE: Testing row cache feature in trunk: write should put record in
 	cache
Thread-Topic: Testing row cache feature in trunk: write should put record in
 	cache
Thread-Index: AcqvkUeqVuYI83mqSWm8w0M90Tu7mAPYz9iQ
Message-ID: 
 <33FDEB0CE2F65F41A4CF8769247BB3668DC377AA2E@EXVMBX016-3.exch016.msoutlookonline.net>
References: <ad2266761002161711t5eb94515y7574acc3e4118a7@mail.gmail.com>
 	<e06563881002161717o6bbeecd2u56f905eb2d86e547@mail.gmail.com>
 	<e06563881002161720j3deb85b5v2a0e222a6e4f5bd9@mail.gmail.com>
 	<ad2266761002161837p5f67ca4ag7b6140046c7e114b@mail.gmail.com>
 	<e06563881002162107o3e2935d2w66fabb7fe24d1b0@mail.gmail.com>
 <e06563881002162122t4a99543g79adb4499050b084@mail.gmail.com>
In-Reply-To: <e06563881002162122t4a99543g79adb4499050b084@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

This is interesting for the use cases I'm looking at Cassandra for, so if t=
hat offer still stands I'll take you up on it. I took a crack at it in http=
s://issues.apache.org/jira/browse/CASSANDRA-860 - also in large part to get=
 my feet wet with the code.=20

-----Original Message-----
From: Jonathan Ellis [mailto:jbellis@gmail.com]=20
Sent: Tuesday, February 16, 2010 9:22 PM
To: cassandra-user@incubator.apache.org
Subject: Re: Testing row cache feature in trunk: write should put record in=
 cache

... 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.)
>>
>>
>