Re: deprecating setLogLevel

From: Mike Kienenberger (mkienen..mail.com)
Date: Sun Sep 18 2005 - 17:07:18 EDT

  • Next message: Kevin Menard: "Running tests"

    Dumping it sounds good to me. I've never used it.

    On 9/18/05, Andrus Adamchik <andru..bjectstyle.org> wrote:
    > Anybody finds these methods useful?
    >
    > DataContext.commitChanges(Level);
    > Query.setLoggingLevel(Level);
    >
    >
    > I thought we won't have to deal with factoring out Log4J in this
    > release cycle, but I hate having bulky Log4J jar on the client, so I
    > am going back to this earlier discussion:
    >
    > http://www.objectstyle.org/cayenne/lists/cayenne-devel/2005/05/0085.html
    >
    > What I want to do now is to deprecate methods above. If we want to
    > keep this functionality in some form, it can be replaced with
    > solution provided by Holger:
    >
    > http://cvs.sourceforge.net/viewcvs.py/cayenne/sandbox/holger/src/org/
    > objectstyle/cayenne/util/
    >
    > However I am inclined to get rid of it all together. Having API
    > control over logging level seemed like a good idea 3 years ago, but
    > now it looks more like bad design. Anyone thinks we should absolutely
    > keep it?
    >
    > .... Now when I was writing this, I actually found a reasonable
    > substitute - QueryLogger.setLoggingLevel(..). We just need it to be
    > stored in a ThreadLocal. So I guess I have my answer?....
    >
    > Andrus
    >
    >
    >
    >



    This archive was generated by hypermail 2.0.0 : Sun Sep 18 2005 - 17:07:21 EDT