Skip to content

Commit 3c28abb

Browse files
bulk: catch another place generating verbose logging
Motivation: ----------- Patch 466b328 addressd the issue #7931 only partially Modification: ------------ Fix another place where the auto clear of requests leads to unnecessary logging Result: ------- Smaller log files. Ticket: #7931 Acked-by: Chris Green Target: trunk Request: 11.1 Request: 11.0 Request: 10.2 Request: 10.1 Request: 10.0 Request: 9.2 Require-book: no Require-notes: yes
1 parent 84dfda3 commit 3c28abb

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

modules/dcache-bulk/src/main/java/org/dcache/services/bulk/handler/BulkRequestHandler.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -318,6 +318,12 @@ private boolean setStateIfTerminated(String id, boolean cancelled) {
318318
statistics.incrementRequestsCompleted();
319319
return true;
320320
}
321+
} catch (BulkRequestNotFoundException e) {
322+
/*
323+
* Request has been cleared already.
324+
*/
325+
LOGGER.debug("setStateIfTerminated, request {} was auto-cleared on termination.", id);
326+
return true;
321327
} catch (BulkServiceException e) {
322328
LOGGER.error("Failed to post-process request {}: {}.", id,
323329
e.toString());
@@ -344,4 +350,4 @@ private boolean storeJobTarget(BulkRequestTarget target) throws BulkServiceExcep
344350

345351
return true;
346352
}
347-
}
353+
}

0 commit comments

Comments
 (0)