added delete_source functionality to the dissect processor#5662
Conversation
Signed-off-by: Zeeshan Alam <Zeeshan.Alam+fidelity@fmr.com>
san81
left a comment
There was a problem hiding this comment.
Nice. Thanks for the contribution.
|
@alamzeeshan , Can you create a GitHub issue against the documentation-website with information for this? |
@dlvenable Created opensearch-project/documentation-website#9951 |
|
I see this failure: You can run the unit tests for this project to fast check on your side: |
Signed-off-by: Zeeshan Alam <Zeeshan.Alam+fidelity@fmr.com>
Hi @dlvenable , I missed one import statement. I have update the code and verified the build using |
Hi @san81 , Can you please re-approve this PR. I added a missing import statement. |
san81
left a comment
There was a problem hiding this comment.
Thank you for this nice contribution
…h-project#5662) * completed feature request Signed-off-by: Zeeshan Alam <Zeeshan.Alam+fidelity@fmr.com> * added missing import statement Signed-off-by: Zeeshan Alam <Zeeshan.Alam+fidelity@fmr.com> --------- Signed-off-by: Zeeshan Alam <Zeeshan.Alam+fidelity@fmr.com> Co-authored-by: Zeeshan Alam <Zeeshan.Alam+fidelity@fmr.com> Signed-off-by: Jonah Calvo <caljonah@amazon.com>
Description
This change will add a new functionality to the
dissectprocessor. Now users can choose a new optiondelete_source: truewhich will delete the source field ifdissectprocessor was successful in extracting the fields.Example config:
Issues Resolved
Resolves #5345
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.