Skip to content

Commit 63af186

Browse files
committed
ThrottleQueueTaskDispatcher.java : shift logging priorities for less spam in FINE level
1 parent 3cadb3a commit 63af186

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/main/java/hudson/plugins/throttleconcurrents/ThrottleQueueTaskDispatcher.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ private boolean isAnotherBuildWithSameParametersRunningOnNode(Node node, Queue.I
339339
List<ParameterValue> itemParams = getParametersFromQueueItem(item);
340340

341341
if (paramsToCompare.size() > 0) {
342-
LOGGER.log(Level.FINE, "filter itemParams " + itemParams +
342+
LOGGER.log(Level.FINER, "filter itemParams " + itemParams +
343343
" (from queue) to only pick up to " + paramsToCompare.size() +
344344
": " + paramsToCompare + " (from throttle config)");
345345
itemParams = doFilterParams(paramsToCompare, itemParams);
@@ -356,7 +356,7 @@ private boolean isAnotherBuildWithSameParametersRunningOnNode(Node node, Queue.I
356356
parentTask.getOwnerTask().getName().equals(item.task.getName())) {
357357
List<ParameterValue> executingUnitParams = getParametersFromWorkUnit(exec.getCurrentWorkUnit());
358358

359-
LOGGER.log(Level.FINE, "filter executingUnitParams" +
359+
LOGGER.log(Level.FINER, "filter executingUnitParams" +
360360
" on " + computer.getDisplayName() + "#" + exec.getNumber() +
361361
" in build (" + exec.getCurrentWorkUnit() + ") from original " +
362362
executingUnitParams + " to only pick up to " +
@@ -391,27 +391,27 @@ private boolean isAnotherBuildWithSameParametersRunningOnNode(Node node, Queue.I
391391
*/
392392
private List<ParameterValue> doFilterParams(List<String> paramsToCompare, List<ParameterValue> OriginalParams) {
393393
if (paramsToCompare.isEmpty()) {
394-
LOGGER.log(Level.FINE, "doFilterParams(): paramsToCompare.isEmpty() => return OriginalParams");
394+
LOGGER.log(Level.FINEST, "paramsToCompare.isEmpty() => return OriginalParams");
395395
return OriginalParams;
396396
}
397397

398398
List<ParameterValue> newParams = new ArrayList<ParameterValue>();
399-
LOGGER.log(Level.FINE, "doFilterParams(): filtering by paramsToCompare = " + paramsToCompare);
399+
LOGGER.log(Level.FINEST, "filtering by paramsToCompare = " + paramsToCompare);
400400

401401
for (ParameterValue p : OriginalParams) {
402-
LOGGER.log(Level.FINE, "doFilterParams(): checking if original " +
402+
LOGGER.log(Level.FINEST, "checking if original " +
403403
p.getName() + " is on our list of paramsToCompare?.." );
404404
if (paramsToCompare.contains(p.getName())) {
405-
LOGGER.log(Level.FINE, "doFilterParams(): we have a hit in OriginalParams: " + p);
405+
LOGGER.log(Level.FINEST, "we have a hit in OriginalParams: " + p);
406406
newParams.add(p);
407407
}
408408
}
409409
if (newParams.size() == 0 ) {
410-
LOGGER.log(Level.WARNING, "doFilterParams(): Error selecting params," +
410+
LOGGER.log(Level.WARNING, "Error selecting params," +
411411
" got no hits of " + paramsToCompare + " in " + OriginalParams +
412412
" : is the job configuration valid?");
413413
} else if (newParams.size() < paramsToCompare.size() ) {
414-
LOGGER.log(Level.WARNING, "doFilterParams(): Error selecting params," +
414+
LOGGER.log(Level.WARNING, "Error selecting params," +
415415
" not all of " + paramsToCompare + " were present in " + OriginalParams +
416416
" : is the job configuration valid?");
417417
}

0 commit comments

Comments
 (0)