[ 
https://issues.apache.org/jira/browse/CASSANDRA-625?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

gabriele renzi updated CASSANDRA-625:
-------------------------------------

    Attachment: CASSANDRA-625-3.patch

heck, I can't understand why I'm failing to build this, but this time it should 
be correct. I'm really sorry for having caused you all a waste of time.

$ svn up
At revision 921255.
f$ svn info
Path: .
URL: http://svn.apache.org/repos/asf/incubator/cassandra/trunk
Repository Root: http://svn.apache.org/repos/asf
Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
Revision: 921255
Node Kind: directory
Schedule: normal
Last Changed Author: eevans
Last Changed Rev: 921220
Last Changed Date: 2010-03-10 04:48:12 +0100 (Wed, 10 Mar 2010)

$ svn diff > CASSANDRA-625-3.patch
$ grep RangeVerb CASSANDRA-625.patch 
$  ok none here...


> Migrate to slf4j from log4j in cassandra code
> ---------------------------------------------
>
>                 Key: CASSANDRA-625
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-625
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>         Environment: all
>            Reporter: gabriele renzi
>            Priority: Minor
>             Fix For: 0.7
>
>         Attachments: CASSANDRA-625-3.patch
>
>
> currently code generated from thrift uses slf4j while cassandra uses raw 
> log4j. 
> There are various level in which this can be changed: 
> * simply switch log4j loggers with slf4j logger (done in patch)
> * replacing log4j-isms (e.g. string concatenation+isFooEnabled in all code 
> paths) with slf4j-isms (marker string + additional arguments)
> * remove code that is already available for lsf4j (LogUtils.java, I believe 
> is unnecessary)
> I am posting this as a reminder, I have a half baked patch locally but it 
> needs more work 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to