ChrisSamo632 commented on pull request #4761: URL: https://github.com/apache/nifi/pull/4761#issuecomment-762779571
Took a quick look at this and actually most of the Scripted processors/controllers/reporting tasks are already covered by the `org.apache.nifi` @ INFO entry in the `logback.xml` files. The `org.apache.nifi.processors` @ WARN is what stopped the `ExecuteGroovyProcessor` from outputting @ INFO hence the first commit. The only other processor I think that should probably have been outputting @ INFO but wouldn't by default is `org.apache.nifi.processors.script.ScriptedTransformRecord` - it contains an INFO log to report how many records it's transformed/dropped for a FlowFile, but this will be getting omitted from logs currently (unless someone updates their config). So I've added `org.apache.nifi.processors.script` @ INFO to capture this (and any other components in the same package, e.g. `ExecuteScript`). N.B. the only `.info()` log I could see in the newly captured classes is the one mentioned above, which would bring `ScriptedTransformRecord` more inline with the default logging for other non-script record processors, e.g. `SplitRecord`. ---------------------------------------------------------------- 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: [email protected]
