Bug 60565

Summary: Migrate LogKit to SLF4J - Optimize logging statements. e.g, message format args, throwable args, unnecessary if-enabled-logging in simple ones, etc.
Product: JMeter Reporter: Woonsan Ko <woonsan>
Component: MainAssignee: JMeter issues mailing list <issues>
Status: RESOLVED FIXED    
Severity: enhancement CC: p.mouawad
Priority: P2    
Version: 3.1   
Target Milestone: ---   
Hardware: All   
OS: Mac OS X 10.1   
Bug Depends on: 60564    
Bug Blocks:    

Description Woonsan Ko 2017-01-09 17:55:27 UTC
As being discussed in the dev thread [1], the second step (after #60564 done) would be to optimize each logging statement. e.g, message format args, throwable args, unnecessary if-enabled-logging in simple ones, etc.

[1] http://markmail.org/thread/tloqa5zmuj26nrqn
Comment 1 Philippe Mouawad 2017-02-11 08:30:53 UTC
Done by PRs 263 to 274
Comment 2 Philippe Mouawad 2017-02-11 08:33:50 UTC
PR Merger: pmouawad
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Author : Wonnsan Ko 

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Date: Sat Feb 11 08:32:11 2017
New Revision: 1782555

URL: http://svn.apache.org/viewvc?rev=1782555&view=rev
Log:
Bug 60565 - Migrate LogKit to SLF4J - Optimize logging statements. e.g, message format args, throwable args, unnecessary if-enabled-logging in simple ones, etc.
Contributed by Woonsan Ko
Done by PRs 263 to 274
Bugzilla Id: 60565

Modified:
    jmeter/trunk/xdocs/changes.xml
Comment 3 Philippe Mouawad 2017-02-11 08:36:07 UTC
(In reply to Philippe Mouawad from comment #2)
> PR Merger: pmouawad
> !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> 
> Author : Wonnsan Ko 
Woonsan Ko (sorry for typo)
> 
> !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> 
> Date: Sat Feb 11 08:32:11 2017
> New Revision: 1782555
> 
> URL: http://svn.apache.org/viewvc?rev=1782555&view=rev
> Log:
> Bug 60565 - Migrate LogKit to SLF4J - Optimize logging statements. e.g,
> message format args, throwable args, unnecessary if-enabled-logging in
> simple ones, etc.
> Contributed by Woonsan Ko
> Done by PRs 263 to 274
> Bugzilla Id: 60565
> 
> Modified:
>     jmeter/trunk/xdocs/changes.xml