Github user MikeThomsen commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2518#discussion_r184733507
--- Diff:
nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseRecord.java
---
@@ -75,6 +83,17 @@
.expressionLanguageSupported(ExpressionLanguageScope.FLOWFILE_ATTRIBUTES)
.addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
.build();
+ protected static final PropertyDescriptor DEFAULT_VISIBILITY_STRING =
new PropertyDescriptor.Builder()
--- End diff --
I'm a little leery about jumping in there because PutHBaseRecord has a
simplistic method of handling complex records, and we might want to change that
down the road. Using `pickVisibilityString` could complicate things.
---
