ijokarumawak commented on a change in pull request #3580: NIFI-6436 Fix NPE at
StandardPublicPort
URL: https://github.com/apache/nifi/pull/3580#discussion_r314218363
##########
File path:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-site-to-site/src/main/java/org/apache/nifi/remote/StandardPublicPort.java
##########
@@ -121,10 +121,12 @@ public StandardPublicPort(final String id, final String
name, final ProcessGroup
@Override
public void reportEvent(final Severity severity, final String
category, final String message) {
- final String groupId = processGroup.getIdentifier();
- final String groupName = processGroup.getName();
+ final String groupId =
StandardPublicPort.this.getProcessGroup().getIdentifier();
Review comment:
@markap14 Thanks for reviewing. I was worrying about the constructor
argument, too. Pushed another commit to remove the constructor arg.
----------------------------------------------------------------
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
With regards,
Apache Git Services
