[
https://issues.apache.org/jira/browse/NIFI-1118?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15201611#comment-15201611
]
Joseph Witt commented on NIFI-1118:
-----------------------------------
Correct me if I'm wrong but 'deprecation' here means "feature removal". If it
were deprecation in that it was still supported but going to be removed later
then I'd be with you.
If this change, as I understand it, is going to break the existing behavior (in
addition to eliminating the bug you point out) then we're doing too much
surgery right now and I'm not in support.
If this simply fixes the bug and truly deprecates (keeps the feature but
indicates it will go away soon) then I am supportive.
Can you please clarify.
Further, if we're only talking about a bug now then let's move it to 0.6.1.
But there are other features you introduced (which are good) such as limiting
line length and filtering header lines. This thing is doing a lot and as I
read it introducing risk.
> Enable SplitText processor to limit line length and filter header lines
> -----------------------------------------------------------------------
>
> Key: NIFI-1118
> URL: https://issues.apache.org/jira/browse/NIFI-1118
> Project: Apache NiFi
> Issue Type: Improvement
> Components: Extensions
> Reporter: Mark Bean
> Assignee: Joe Skora
> Fix For: 0.6.0
>
>
> Include the following functionality to the SplitText processor:
> 1) Maximum size limit of the split file(s)
> A new split file will be created if the next line to be added to the current
> split file exceeds a user-defined maximum file size
> 2) Header line marker
> User-defined character(s) can be used to identify the header line(s) of the
> data file rather than a predetermined number of lines
> These changes are additions, not a replacement of any property or behavior.
> In the case of header line marker, the existing property "Header Line Count"
> must be zero for the new property and behavior to be used.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)