Lehel44 commented on a change in pull request #5093:
URL: https://github.com/apache/nifi/pull/5093#discussion_r638576761
##########
File path:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/tasks/ConnectableTask.java
##########
@@ -251,8 +251,8 @@ public InvocationResult invoke() {
} catch (final Throwable t) {
// Use ComponentLog to log the event so that a bulletin will
be created for this processor
final ComponentLog procLog = new
SimpleProcessLogger(connectable.getIdentifier(),
connectable.getRunnableComponent());
- procLog.error("{} failed to process session due to {};
Processor Administratively Yielded for {}",
- new Object[] {connectable.getRunnableComponent(), t,
schedulingAgent.getAdministrativeYieldDuration()}, t);
+ procLog.error("Failed to process session due to {}; Processor
Administratively Yielded for {}",
+ new Object[] {t,
schedulingAgent.getAdministrativeYieldDuration()}, t);
Review comment:
Yes, this is an additional fix, I mentioned it in the description of the
PR. The component name is automatically added by the logger, there's no need to
add it here, it will occur twice.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
