Github user trkurc commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/1941#discussion_r125165016
  
    --- Diff: 
nifi-nar-bundles/nifi-snmp-bundle/nifi-snmp-processors/src/main/java/org/apache/nifi/snmp/processors/SNMPUtils.java
 ---
    @@ -191,26 +164,6 @@ private static void 
addAttributeFromVariable(VariableBinding variableBinding, Ma
         }
     
         /**
    -     * Will validate if provided name corresponds to valid SNMP property.
    -     * @param name the name of the property
    -     * @return 'true' if valid otherwise 'false'
    -     */
    -    public static boolean isValidSnmpPropertyName(String name) {
    --- End diff --
    
    I know this method isn't used in your improved attribute populator, but it 
shouldn't be removed as it is a public method in a util class. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to