Hello Alexei,

It would be nice to add something like as follows:
    boolean isInfoEnabled();
    boolean isDebugEnabled();
or
    boolean isLoggable(Level) - the same way which System.Logger suggests

Thanks,
S.

пт, 26 мар. 2021 г. в 17:41, Alexei Scherbakov <alexey.scherbak...@gmail.com
>:

> Andrey,
>
> I've introduced a new class LogWrapper to fix usability issues [1]
>
> The suggested usage is something like:
>
> private static LogWrapper LOG = new LogWrapper(MyClass.class);
>
> [1]
>
> https://github.com/gridgain/apache-ignite-3/blob/9acb050a6a6a601ead849797293a1d0ad48ab9e0/modules/core/src/main/java/org/apache/ignite/lang/LogWrapper.java
>
> пт, 26 мар. 2021 г. в 16:05, Andrey Mashenkov <andrey.mashen...@gmail.com
> >:
>
> > Forgot to attach a link to the PR with an example [1].
> >
> > [1] https://github.com/apache/ignite-3/pull/59
> >
> > On Fri, Mar 26, 2021 at 4:03 PM Andrey Mashenkov <
> > andrey.mashen...@gmail.com>
> > wrote:
> >
> > > Hi Igniters,
> > >
> > > In almost every new task we faced the problem of what logger has to be
> > > used: JUL. log4J or any else.
> > >
> > > Since JDK 9 there is a System.Logger which interface looks acceptable
> for
> > > use,
> > > excepts maybe some usability issues like method signatures.
> > > LogLevel is passed as a mandatory argument, and no shortcut methods are
> > > provided (like 'warn', 'error' or 'info').
> > >
> > > I like Alex Scherbakov idea [1] to use a brand new JDK system logger by
> > > default and
> > > extend it with shortcut methods.
> > >
> > > I've created a ticket to unify logger usage in Ignite-3.0 project to
> fix
> > > already existed code.
> > >
> > > Any thoughts or objections?
> > >
> > > --
> > > Best regards,
> > > Andrey V. Mashenkov
> > >
> >
> >
> > --
> > Best regards,
> > Andrey V. Mashenkov
> >
>
>
> --
>
> Best regards,
> Alexei Scherbakov
>

Reply via email to