Mailing-List: contact commons-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Received-SPF: pass (hermes.apache.org: domain of oliver.zeigermann@gmail.com
 designates 64.233.184.196 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:reply-to:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:references;
        b=YirwIZHx/9lXOtk16wNBvLLqE3MXZeO6aSIg9/t7Xp9rA7YmBttqwaGdneL9kdtlHzmVmHVoPYhr7slMFRkGsNpkg7GNm5IimjmfAMyUQyp7u/1vtXDIKWHNv+ikac2240rWWlROYtgTkGU06nr1R6U4RaHKYEtJjDHbIu6jp/0=
Message-ID: <9da4f45204112903393fbb60a5@mail.gmail.com>
Date: Mon, 29 Nov 2004 12:39:53 +0100
From: Oliver Zeigermann <oliver.zeigermann@gmail.com>
Reply-To: ozeigermann@apache.org
To: Jakarta Commons Developers List <commons-dev@jakarta.apache.org>
Subject: Re: [transaction] improved logging interface.
In-Reply-To: <41AAEEF0.3060000@apache.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
References: <41A5DBD1.5060109@apache.org>
	 <9da4f452041126145356703089@mail.gmail.com>
	 <41AAEEF0.3060000@apache.org>

On Mon, 29 Nov 2004 10:42:08 +0100, Stefan L=FCtzkendorf
<luetzkendorf@apache.org> wrote:
> Ok, this would do it too.
>=20
> The other way would it make nicer, but you are right it is not required
> absolutly. (One advantage would be that the parametrized version coulkd
> use a more efficient type of string processing)

If we are logging finest performance wouldn't be our major goal and if
we are not the string concatenation wouldn't be done in the first
place. So IMHO there is little need for a parameterized version, is
there?
=20
> Than we should do the isXXXEnabled tests propertly. Because the most
> logging is "fine", "finer" or "finest" and we sould avoid this in an
> working environment.

Agreed!

Oliver
=20
> I'll do this next time.
>=20
> Regards, Stefan
>=20
>=20
>=20
> Oliver Zeigermann wrote:
>=20
> > I thought the standard way to handle this is to guard the log
> > statements with checks if the log level is enabled like:
> >
> > if (isFineEnabled())
> >   logFine(anyObject1 + "any string" + anyObject2);
> >
> > which already is possible with LoggerFacade. This would make the
> > extension obsolete...
> >
> > Oliver
> >
> > On Thu, 25 Nov 2004 14:19:13 +0100, Stefan L=FCtzkendorf
> > <luetzkendorf@apache.org> wrote:
> >
> >>Hello Oliver,
> >>
> >>I would like to improve the LoggerFacade interface with methods like,
> >>
> >>logFiner(String message, Object param1);
> >>logFiner(String message, Object param1, Object param2);
> >>logFine(String message, Object param1);
> >>...
> >>
> >>to avoid the frequently used calls like
> >>logFine(anyObject1 + "any string" + anyObject2);
> >>
> >>This would reduce the overhead if logging is disabled.
> >>
> >>Changing the interfaces before the first release would be good, I think=
.
> >>What do you think?
> >>
> >>Regards, Stefan
> >>
> >>--
> >>Stefan L=FCtzkendorf  --  luetzkendorf@apache.org
> >>
> >>---------------------------------------------------------------------
> >>To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> >>For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> >>
> >>
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> >
> >
>=20
> --
>=20
>=20
> Stefan L=FCtzkendorf  --  luetzkendorf@apache.org
>=20
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
>=20
>

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org