[ 
https://issues.apache.org/jira/browse/IGNITE-7284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16331577#comment-16331577
 ] 

Valentin Kulichenko commented on IGNITE-7284:
---------------------------------------------

[~slukyanov], I disagree about system property caching. I don't see much value 
in checking it every time the warning is added to the log. If we want to add an 
ability to change this value dynamically, there should be another way, probably 
via MBean. Feel free to create a separate ticket for this. Agree on the rest 
though, so I made the change myself and merged code to master. Thanks for the 
contribution!

> Introduce DEV_ONLY marker to IgniteLogger
> -----------------------------------------
>
>                 Key: IGNITE-7284
>                 URL: https://issues.apache.org/jira/browse/IGNITE-7284
>             Project: Ignite
>          Issue Type: Improvement
>          Components: general
>    Affects Versions: 2.3
>            Reporter: Valentin Kulichenko
>            Assignee: Valentin Kulichenko
>            Priority: Major
>             Fix For: 2.5
>
>
> Need to add markers support to {{IgniteLogger}} and then introduce 
> {{DEV_ONLY}} marker for warnings that can be suppressed in prod environments.
> Problem and solution are discussed in more detail here: 
> http://apache-ignite-developers.2346864.n4.nabble.com/Suppressing-quot-development-quot-warning-td25195.html
> Make sure to update Loggers documentation explaining how the parameter works 
> and what's it for:
> https://apacheignite.readme.io/docs/logging



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

Reply via email to