You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by "Stanilovsky Evgeny (JIRA)" <ji...@apache.org> on 2018/06/06 14:15:00 UTC

[jira] [Created] (IGNITE-8724) Skip logging 3-rd parameter while calling U.warn with initialized logger.

Stanilovsky Evgeny created IGNITE-8724:
------------------------------------------

             Summary: Skip logging 3-rd parameter while calling U.warn with initialized logger.
                 Key: IGNITE-8724
                 URL: https://issues.apache.org/jira/browse/IGNITE-8724
             Project: Ignite
          Issue Type: Bug
          Components: general
    Affects Versions: 2.5
            Reporter: Stanilovsky Evgeny
            Assignee: Stanilovsky Evgeny


There are a lot of places where exception need to be logged, for example :

{code:java}
U.warn(log,"Unable to await partitions release future", e);
{code}

but current U.warn realization silently swallow it.

{code:java}
    public static void warn(@Nullable IgniteLogger log, Object longMsg, Object shortMsg) {
        assert longMsg != null;
        assert shortMsg != null;

        if (log != null)
            log.warning(compact(longMsg.toString()));
        else
            X.println("[" + SHORT_DATE_FMT.format(new java.util.Date()) + "] (wrn) " +
                compact(shortMsg.toString()));
    }
{code}

fix, looks like simple add:

{code:java}
    public static void warn(@Nullable IgniteLogger log, Object longMsg, Throwable ex) {
{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)