Skip to content

add ignore_missing to split pipeline processor #20840

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
djschny opened this issue Oct 10, 2016 · 3 comments
Closed

add ignore_missing to split pipeline processor #20840

djschny opened this issue Oct 10, 2016 · 3 comments
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >enhancement v5.1.1 v6.0.0-alpha1

Comments

@djschny
Copy link
Contributor

djschny commented Oct 10, 2016

Elasticsearch version: v5.0.0-rc1

Description of the problem including expected versus actual behavior:

The split processor throws an error if the json element is not present in the document. Request to have it implement ignore_missing to resolve like what was done for other processors.

See #19995 for reference.
@djschny
Copy link
Contributor Author

djschny commented Oct 11, 2016

@talevy and @eskibars when you get a chance would you mind looking at this please?

@talevy talevy added :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >enhancement v6.0.0-alpha1 v5.1.1 labels Oct 11, 2016
@talevy talevy self-assigned this Oct 11, 2016
@talevy talevy mentioned this issue Oct 17, 2016
Merged
@talevy
Copy link
Contributor

talevy commented Oct 17, 2016

@djschny apologies for the delay (I know I lied to you about how soon this would be done 😸)

let me know what you think!

@djschny
Copy link
Contributor Author

djschny commented Oct 17, 2016

Looks good from what I can tell. Thank you!

talevy added a commit to talevy/elasticsearch that referenced this issue Oct 18, 2016
@talevy
add ignore_missing option to SplitProcessor
8106853 
Closes elastic#20840.
talevy added a commit that referenced this issue Nov 16, 2016
@talevy
add ignore_missing option to SplitProcessor (#20982)
6796464 
Closes #20840.
talevy added a commit that referenced this issue Nov 16, 2016
@talevy
add ignore_missing option to SplitProcessor (#20982)
6f5ff52 
Closes #20840.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >enhancement v5.1.1 v6.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@djschny @talevy