Github user robertrbruno commented on the issue:
https://github.com/apache/nifi/pull/2501
@MikeThomsen changes look good to me.
---
Github user robertrbruno commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2501#discussion_r174307349
--- Diff:
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch
Github user robertrbruno commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2501#discussion_r174305856
--- Diff:
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch
Github user robertrbruno commented on the issue:
https://github.com/apache/nifi/pull/2501
Gave your addition a quick look. You should be fine, but I believe
unnecessary since the writevalue function will never be called when the value
is "null" due to the continue