mosermw commented on code in PR #7922:
URL: https://github.com/apache/nifi/pull/7922#discussion_r1384091419
##########
nifi-nar-bundles/nifi-jslt-bundle/nifi-jslt-processors/src/main/java/org/apache/nifi/processors/jslt/JSLTTransformJSON.java:
##########
@@ -107,6 +108,16 @@ public class JSLTTransformJSON extends AbstractProcessor {
.defaultValue(EACH_OBJECT.getValue())
.build();
+ public static final PropertyDescriptor RESULT_FILTER = new
PropertyDescriptor.Builder()
+ .name("jslt-transform-result-filter")
+ .displayName("Transform Result Filter")
+ .description("An optional filter of output results using another
JSLT, allowing you to change the default filter."
+ + " The default filter is \". != null and . != {} and . !=
[]\" which excludes objects with null values, empty objects and empty arrays")
Review Comment:
@exceptionfactory I made the new property required, gave it a default value,
and updated the description. My performance tests showed a small penalty
(roughly 5%) using a simple JSLT filter over the built-in default. To mitigate
this, if the property is set to the default filter value, then I use the
library's built-in default filter. This should preserve current performance.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
