Mailing-List: contact log4j-user-help@logging.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Log4J Users List" <log4j-user@logging.apache.org>
Received-SPF: pass (nike.apache.org: domain of boards@gmail.com designates
 209.85.218.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAPvyOri5sQduTteXD0jtngx+VwzY0NGHw8JsbiWECEW1XCbwcA@mail.gmail.com>
References: 
 <CAPvyOrhuxcNifNea+HOKXy5O9ATo3PP0O+3-GeSE45CiCzc4yg@mail.gmail.com>
	<CACmp6kqi2N3oAv1A0RWB2VHEk862uckE9yz5iPcjjDqfhWXewQ@mail.gmail.com>
	<CAPvyOrh2fK9KdAWKm232wnYW36ha8-rNkqAogu+3hn1Xpux8yg@mail.gmail.com>
	<D164C8D0-3AAC-4945-8335-0851A2782F5A@dslextreme.com>
	<CAPvyOriKTnroaKKJrVpNAkHvef91G1020L1URTaS4Yr943SWFQ@mail.gmail.com>
	<CAAFrFhT4n4tT0w3u36twVWDeh_uRFro7Hwr_pepKjOaY9CG99Q@mail.gmail.com>
	<CAPvyOrj_e4uSWVLYr15wU0TxLfCwuYUxKXNh+DMQPzfF4HwRxA@mail.gmail.com>
	<05D82BA5-846E-45D8-9256-CF06C01D2B46@gmail.com>
	<CAPvyOrg+QhgV58gG96Wx4eoyDmq7t_6NywJnY=veNPROAcjkEQ@mail.gmail.com>
	<CAAFrFhRh8jZ5e=U8Djj38YDTdEBEzfOkUisq9DgLdcquWOc8ow@mail.gmail.com>
	<CAPvyOrguZC8qEoafqcFF2Z64bP1LAR94OeLXgSfN3ipj63x8ow@mail.gmail.com>
	<CAPvyOri5sQduTteXD0jtngx+VwzY0NGHw8JsbiWECEW1XCbwcA@mail.gmail.com>
Date: Tue, 2 Sep 2014 19:01:47 -0500
Message-ID: 
 <CACmp6kqtT=0gAh24yz_+0ZEvSErYisvxYHO3+dQejrCcT8oB_w@mail.gmail.com>
Subject: Re: Log4j2 with SL4J Adapter
From: Matt Sicker <boards@gmail.com>
To: Log4J Users List <log4j-user@logging.apache.org>
Content-Type: multipart/alternative; boundary=001a11c2eb9c35394705021df18b

--001a11c2eb9c35394705021df18b
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

If you prefer using GitHub, you can file a pull request instead of a patch
file. I still want to test out the functionality, but it should work just
fine.


On 2 September 2014 09:29, Yogesh Rao <yogu13@gmail.com> wrote:

> Hi,
>
> Just an update on this... i had little time last week to work on this
> issue. I am looking into this now and checking if a sandbox can be create=
d
> for execution of the test cases using class loaders.
>
> Thank you!
>
> Regards,
> -Yogesh
>
>
> On Mon, Aug 25, 2014 at 10:48 AM, Yogesh Rao <yogu13@gmail.com> wrote:
>
> > Sure Remko! Will try for it.
> >
> > Thank you!
> >
> > Regards,
> > -Yogesh
> >
> > On Monday, August 25, 2014, Remko Popma <remko.popma@gmail.com> wrote:
> >
> >> Great, thank you! I'll have limited PC access the next ten days or so.
> >> If you want to take a stab at providing a patch that includes both a
> JUnit
> >> test and a fix, go for it!
> >>
> >>
> >> On Sun, Aug 24, 2014 at 11:31 PM, Yogesh Rao <yogu13@gmail.com> wrote:
> >>
> >> > Defect logged under LOG4J2-796
> >> >
> >> > Regards,
> >> > -Yogesh
> >> >
> >> >
> >> > On Sun, Aug 24, 2014 at 2:41 PM, Remko Popma <remko.popma@gmail.com>
> >> > wrote:
> >> >
> >> > > Yogesh,
> >> > >
> >> > > Thanks! Both would be really helpful!
> >> > >
> >> > > Remko
> >> > >
> >> > > Sent from my iPhone
> >> > >
> >> > > > On 2014/08/24, at 9:09, Yogesh Rao <yogu13@gmail.com> wrote:
> >> > > >
> >> > > > Thanks Remko!
> >> > > >
> >> > > > would you like me to create a JIRA ticket for the same ? I could
> >> also
> >> > try
> >> > > > giving a shot at creating that unit test case for you..
> >> > > >
> >> > > > Regards,
> >> > > > -Yogesh
> >> > > >
> >> > > >
> >> > > >> On Sun, Aug 24, 2014 at 12:35 PM, Remko Popma <
> >> remko.popma@gmail.com>
> >> > > wrote:
> >> > > >>
> >> > > >> This looks like a bug. The output is the FQCN (fully qualified
> >> name of
> >> > > the
> >> > > >> logger).
> >> > > >> This is probably introduced by the changes made for LOG4J2-555.
> >> > > >>
> >> > > >> Our unit tests are not picking this up because they use
> >> > > >> the ch.qos.logback.classic.Logger, which is a LocationAwareLogg=
er
> >> and
> >> > > >> follows a different code path.
> >> > > >> I suspect that Simple Logger is not LocationAware.
> >> > > >>
> >> > > >> I suspect we should change (in SLF4JLogger.logMessage)
> >> > > >>     logger.debug(fqcn, getMarker(marker),
> >> > message.getFormattedMessage(),
> >> > > >> message.getParameters(), t);
> >> > > >> to
> >> > > >>     logger.debug(getMarker(marker),
> message.getFormattedMessage(),
> >> > > >> message.getParameters(), t);
> >> > > >>
> >> > > >> I can change this but want to make a failing unit test first an=
d
> I
> >> > > haven't
> >> > > >> had time to figure out how to do that yet.
> >> > > >>
> >> > > >>
> >> > > >>
> >> > > >>> On Sun, Aug 24, 2014 at 12:00 PM, Yogesh Rao <yogu13@gmail.com=
>
> >> > wrote:
> >> > > >>>
> >> > > >>> Hi Ralph,
> >> > > >>>
> >> > > >>> The issue for me is log4j2 to slf4j adaptor doesnt seem to be
> >> > working.
> >> > > I
> >> > > >>> added the log4j configuration only when the logger wasnt worki=
ng
> >> > hence
> >> > > i
> >> > > >>> mentioned in brackets that i wasnt sure if its needed or not..
> >> > > >>>
> >> > > >>> So since you mentioned that things should be working without t=
he
> >> > > >>> configuration i removed it and ran but it still doesnt log the
> >> > details
> >> > > as
> >> > > >>> desired.
> >> > > >>>
> >> > > >>> Is this an issue with log4j2?
> >> > > >>>
> >> > > >>> Regards,
> >> > > >>> -Yogesh
> >> > > >>>
> >> > > >>> On Sunday, August 24, 2014, Ralph Goers <
> >> ralph.goers@dslextreme.com>
> >> > > >>> wrote:
> >> > > >>>
> >> > > >>>> I am really not clear on what the issue is. It seems you have
> >> things
> >> > > >>>> configured so that you can write to the Log4j 2 API but
> actually
> >> use
> >> > > >>> SLF4J
> >> > > >>>> for logging.  There is nothing wrong with that but if you are
> >> > routing
> >> > > >> to
> >> > > >>>> SLF4J and using SLF4J Simple Logger then the Log4j 2
> >> configuration
> >> > > >> won=E2=80=99t
> >> > > >>>> even be used.
> >> > > >>>>
> >> > > >>>> What are you really wanting to do and what are you expecting =
to
> >> > > happen?
> >> > > >>>>
> >> > > >>>> Ralph
> >> > > >>>>
> >> > > >>>> On Aug 22, 2014, at 7:57 PM, Yogesh Rao <yogu13@gmail.com
> >> > > >>> <javascript:;>>
> >> > > >>>> wrote:
> >> > > >>>>
> >> > > >>>>> Thanks matt!
> >> > > >>>>>
> >> > > >>>>> First is what i am attempting to check.. Here instead of
> >> logback i
> >> > > >> have
> >> > > >>>>> used slf4j-simple.
> >> > > >>>>>
> >> > > >>>>> Based on this is there any issue with my implementation?
> >> > > >>>>>
> >> > > >>>>> Regards,
> >> > > >>>>> -Yogesh
> >> > > >>>>>
> >> > > >>>>> On Friday, August 22, 2014, Matt Sicker <boards@gmail.com
> >> > > >>> <javascript:;>>
> >> > > >>>> wrote:
> >> > > >>>>>
> >> > > >>>>>> Using log4j-to-slf4j is for when you want to use log4j-api
> but
> >> > with
> >> > > >> an
> >> > > >>>>>> slf4j implementation (like logback). log4j-slf4j-impl would
> >> allow
> >> > > >> you
> >> > > >>> to
> >> > > >>>>>> use slf4j-api and have that log to log4j-core.
> >> > > >>>>>>
> >> > > >>>>>>
> >> > > >>>>>> On 22 August 2014 08:27, Yogesh Rao <yogu13@gmail.com
> >> > > >> <javascript:;>
> >> > > >>>> <javascript:;>>
> >> > > >>>>>> wrote:
> >> > > >>>>>>
> >> > > >>>>>>> Hi,
> >> > > >>>>>>>
> >> > > >>>>>>> I am trying out the API side of log4j2 with the SLF4J
> adaptor.
> >> > With
> >> > > >>> the
> >> > > >>>>>>> sample code written i couldn't see the logging being done
> >> > properly.
> >> > > >>>>>> Perhaps
> >> > > >>>>>>> i am doing something wrong here.
> >> > > >>>>>>>
> >> > > >>>>>>> Firstly versions of the software are :-
> >> > > >>>>>>>
> >> > > >>>>>>> 1. JDK 1.8.x
> >> > > >>>>>>> 2. Log4j2 (2.0.1)
> >> > > >>>>>>> 3. SLF4J 1.7.7
> >> > > >>>>>>> 4. SLF4J's Simple Logger (1.7.7)
> >> > > >>>>>>>
> >> > > >>>>>>> Code written is as under
> >> > > >>>>>>>
> >> > > >>>>>>> HelloWorld.java
> >> > > >>>>>>> --------------
> >> > > >>>>>>>
> >> > > >>>>>>> import org.apache.logging.log4j.LogManager;
> >> > > >>>>>>> import org.apache.logging.log4j.Logger;
> >> > > >>>>>>>
> >> > > >>>>>>>
> >> > > >>>>>>> public class HelloWorld {
> >> > > >>>>>>>
> >> > > >>>>>>> private static final Logger logger =3D
> >> > > >>>> LogManager.getLogger("HelloWorld");
> >> > > >>>>>>>   public static void main(String[] args) {
> >> > > >>>>>>>       logger.error("Hello, World!");
> >> > > >>>>>>>       logger.error("Logging in user {} with birthday {}",
> >> "Ted",
> >> > > >>>>>>> "1/10/1900");
> >> > > >>>>>>>   }
> >> > > >>>>>>>
> >> > > >>>>>>> }
> >> > > >>>>>>>
> >> > > >>>>>>> Log4j.xml (Not sure if this is needed or not)
> >> > > >>>>>>> -------------
> >> > > >>>>>>>
> >> > > >>>>>>> <?xml version=3D"1.0" encoding=3D"UTF-8"?>
> >> > > >>>>>>> <Configuration status=3D"WARN">
> >> > > >>>>>>> <Appenders>
> >> > > >>>>>>>   <Console name=3D"Console" target=3D"SYSTEM_OUT">
> >> > > >>>>>>>     <PatternLayout pattern=3D"%d{HH:mm:ss.SSS} [%t] %-5lev=
el
> >> > > >>> %logger{36}
> >> > > >>>>>> -
> >> > > >>>>>>> %msg%n"/>
> >> > > >>>>>>>   </Console>
> >> > > >>>>>>> </Appenders>
> >> > > >>>>>>> <Loggers>
> >> > > >>>>>>>   <Root level=3D"INFO">
> >> > > >>>>>>>     <AppenderRef ref=3D"Console"/>
> >> > > >>>>>>>   </Root>
> >> > > >>>>>>> </Loggers>
> >> > > >>>>>>> </Configuration>
> >> > > >>>>>>>
> >> > > >>>>>>>
> >> > > >>>>>>> The output generated is
> >> > > >>>>>>> -----------------------------
> >> > > >>>>>>> [main] ERROR HelloWorld -
> >> > > >> org.apache.logging.log4j.spi.AbstractLogger
> >> > > >>>>>>> [main] ERROR HelloWorld -
> >> > > >> org.apache.logging.log4j.spi.AbstractLogger
> >> > > >>>>>>>
> >> > > >>>>>>> Jars in classpath
> >> > > >>>>>>> ----------------------
> >> > > >>>>>>> log4j-api-2.0.1.jar
> >> > > >>>>>>> log4j-to-slf4j-2.0.1.jar
> >> > > >>>>>>> slf4j-api-1.7.7
> >> > > >>>>>>> slf4j-simple-1.7.7
> >> > > >>>>>>>
> >> > > >>>>>>>
> >> > > >>>>>>> Also the code works fine if i remove the slf4j related jar=
s
> >> and
> >> > > >>> replace
> >> > > >>>>>> it
> >> > > >>>>>>> with log4j core.
> >> > > >>>>>>>
> >> > > >>>>>>> Is there any configuration that i have missed ?
> >> > > >>>>>>>
> >> > > >>>>>>> Thank You!
> >> > > >>>>>>>
> >> > > >>>>>>> Regards,
> >> > > >>>>>>> -Yogesh
> >> > > >>>>>>
> >> > > >>>>>>
> >> > > >>>>>>
> >> > > >>>>>> --
> >> > > >>>>>> Matt Sicker <boards@gmail.com <javascript:;> <javascript:;>=
>
> >> > > >>>>
> >> > > >>>>
> >> > > >>>>
> >> > --------------------------------------------------------------------=
-
> >> > > >>>> To unsubscribe, e-mail:
> >> log4j-user-unsubscribe@logging.apache.org
> >> > > >>>> <javascript:;>
> >> > > >>>> For additional commands, e-mail:
> >> log4j-user-help@logging.apache.org
> >> > > >>>> <javascript:;>
> >> > > >>
> >> > >
> >> > >
> ---------------------------------------------------------------------
> >> > > To unsubscribe, e-mail: log4j-user-unsubscribe@logging.apache.org
> >> > > For additional commands, e-mail: log4j-user-help@logging.apache.or=
g
> >> > >
> >> > >
> >> >
> >>
> >
>


--=20
Matt Sicker <boards@gmail.com>

--001a11c2eb9c35394705021df18b--