Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/jenkins
  Commit: a9eafdb973d0e9697e2f7a8a3f4e3441464398f4
      
https://github.com/jenkinsci/jenkins/commit/a9eafdb973d0e9697e2f7a8a3f4e3441464398f4
  Author: Jeff Thompson <37345299+jeffre...@users.noreply.github.com>
  Date:   2018-05-04 (Fri, 04 May 2018)

  Changed paths:
    M core/src/main/java/hudson/model/FingerprintCleanupThread.java
    A core/src/test/java/hudson/model/FingerprintCleanupThreadTest.java

  Log Message:
  -----------
  [JENKINS-50412] Fingerprints logs should be less verbose (#3373)

* JENKINS-50412 - Fingerprints logs should be less verbose

Eliminate the logging line that says
possibly trimming /var/jenkins_home/fingerprints/[...]
This line fills up the fingerprints log file with repetitive, useless 
information.
The operation it calls provides its own logging that can be turned on if 
detailed
logging is actually needed instead of running this one repeatedly.

Also, while in the area, convert a couple of FileFilter inner classes to lambdas
and inline them. This reduces unnecessary boilerplate and improves clarity.

In order to make this change testable, I elected to remove the final 
restriction on the
class and annotate the class as restricted. This allows me to create unit tests 
for
this change and also verify existing capability at the appropriate level.

* Watch out for varying line endings.

Use a reliable check.

* Correct a few minor issues caught in review.

* Change to correct access modifier.



      **NOTE:** This service been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to