Thanks for that, Daniel.

I'm pretty heads down finishing off the last 0.6 issues right now, but
this is on my list to get to.

On Mon, Mar 8, 2010 at 1:25 PM, Daniel Kluesing <d...@bluekai.com> wrote:
> This is interesting for the use cases I'm looking at Cassandra for, so if 
> that offer still stands I'll take you up on it. I took a crack at it in 
> https://issues.apache.org/jira/browse/CASSANDRA-860 - also in large part to 
> get my feet wet with the code.
>
> -----Original Message-----
> From: Jonathan Ellis [mailto:jbel...@gmail.com]
> 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 <jbel...@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 <weiju...@gmail.com> wrote:
>>> Just tried to make quick change to enable it but it didn't work out :-(
>>>
>>>                ColumnFamily cachedRow = cfs.getRawCachedRow(mutation.key());
>>>
>>>                 // What I modified
>>>                 if( cachedRow == null ) {
>>>                     cfs.cacheRow(mutation.key());
>>>                     cachedRow = cfs.getRawCachedRow(mutation.key());
>>>                 }
>>>
>>>                 if (cachedRow != null)
>>>                     cachedRow.addAll(columnFamily);
>>>
>>> How can I open a ticket for you to make the change (enable row cache write
>>> through with an option)?
>>>
>>> Thanks,
>>> -Weijun
>>>
>>> On Tue, Feb 16, 2010 at 5:20 PM, Jonathan Ellis <jbel...@gmail.com> wrote:
>>>>
>>>> On Tue, Feb 16, 2010 at 7:17 PM, Jonathan Ellis <jbel...@gmail.com> wrote:
>>>> > On Tue, Feb 16, 2010 at 7:11 PM, Weijun Li <weiju...@gmail.com> wrote:
>>>> >> Just started to play with the row cache feature in trunk: it seems to
>>>> >> 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.  So on a fresh
>>>> > 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.)
>>>
>>>
>>
>

Reply via email to