[ https://issues.apache.org/jira/browse/NIFI-4136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16200628#comment-16200628 ]
ASF GitHub Bot commented on NIFI-4136: -------------------------------------- Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1955#discussion_r144079474 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/src/main/java/org/apache/nifi/grok/GrokReader.java --- @@ -74,6 +74,8 @@ "The line of text that does not match the Grok Expression will be appended to the last field of the prior message."); static final AllowableValue SKIP_LINE = new AllowableValue("skip-line", "Skip Line", "The line of text that does not match the Grok Expression will be skipped."); + static final AllowableValue THROW_ERROR = new AllowableValue("throw-error", "Error", + "The processing of the flow file containing the line of text will throw an error and will be routed to the approriate relationship."); --- End diff -- Nitpick, "throw an error" might be too dev-centric, perhaps "issue an error"? Not necessary just a suggestion. Also there is a typo in "appropriate", and the PR won't currently merge cleanly, do you mind doing a rebase? Please and thanks! > GrokReader - Add a failure option to unmatch behavior options > ------------------------------------------------------------- > > Key: NIFI-4136 > URL: https://issues.apache.org/jira/browse/NIFI-4136 > Project: Apache NiFi > Issue Type: Improvement > Components: Extensions > Reporter: Pierre Villard > Assignee: Pierre Villard > > At the moment, when using the GrokReader, if a line does not match the grok > expression (and is not part of a stack trace), the line can be either ignored > (the line will be completely skipped) or appended to the last field from the > previous line. > In the case where appending is not desired and that data should not be > ignored/deleted, we should add the option to route the full flow file to the > failure relationship. This way the flow file could be treated in a different > way (for example with SplitText and ExtractGrok to isolate the incorrect > lines and re-route the correct lines back to the Record processors). -- This message was sent by Atlassian JIRA (v6.4.14#64029)