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

    https://github.com/apache/nifi/pull/2256#discussion_r150375690
  
    --- Diff: 
nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseRecord.java
 ---
    @@ -127,6 +127,18 @@
                 .defaultValue("1000")
                 .build();
     
    +    protected static final AllowableValue NULL_FIELD_EMPTY = new 
AllowableValue("empty-string", "Empty String", "Use an empty string");
    --- End diff --
    
    Shouldn't the name of this option be 'Empty Bytes'? As there's Record data 
types other than String, 'Empty Bytes' sounds more appropriate to me.


---

Reply via email to