Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1358#discussion_r93988247
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/SplitText.java
---
@@ -288,7 +290,7 @@ public void process(InputStream in) throws IOException {
int fragmentIndex = 1; // set to 1 to preserve the existing
behavior *only*. Perhaps should be deprecated to follow the 0,1,2... scheme
String fragmentId = UUID.randomUUID().toString();
- if (computedSplitsInfo.size() == 0) {
+ if ((computedSplitsInfo.size() == 0) && (headerFlowFile != null)) {
--- End diff --
Thanks @mattyb149 The part of processor description is added since ver
1.1.0 by NIFI-2851. So I thought @olegz has some reason for that. As I
mentioned within the previous comment, ver 1.0.0 and earlier don't create that
header only flow files.
With a situation that you mentioned, I'd like to have no output from
SplitText to split relationship. If it outputs empty flow files but only with
header, I'd have to filter out those files in downstream flow.
Maybe this should be discussed separately from this PR. Since this PR fixes
its target issue, I'll merge this into master.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
