Skip to content

chore(remap): Restore changes to parse_grok#6488

Merged
pablosichert merged 1 commit intomasterfrom
pablosichert/restore-parse_grok-changes
Feb 18, 2021
Merged

chore(remap): Restore changes to parse_grok#6488
pablosichert merged 1 commit intomasterfrom
pablosichert/restore-parse_grok-changes

Conversation

@pablosichert
Copy link
Contributor

@pablosichert pablosichert commented Feb 18, 2021

Part of #6476.

Note that this only updates the commented unit tests to the state they were in before the merge of #6353.

https://github.com/timberio/vector/blob/d4ccc81c9aa0e82dbac1da26602320ca0c549728/lib/remap-functions/src/parse_grok.rs#L118-L247

The relevant changes to the code itself have been migrated correctly.

Signed-off-by: Pablo Sichert <mail@pablosichert.com>
@pablosichert pablosichert requested review from a team, JeanMertz, StephenWakely and juchiast and removed request for a team and juchiast February 18, 2021 20:51
@pablosichert pablosichert changed the title Restore changes to parse_grok chore(remap): Restore changes to parse_grok Feb 18, 2021
Copy link
Contributor

@StephenWakely StephenWakely left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably could get a more comprehensive typedef out of this function since we know the pattern at compile time. But that is out of scope for this PR.

@pablosichert pablosichert merged commit 58733e3 into master Feb 18, 2021
@pablosichert pablosichert deleted the pablosichert/restore-parse_grok-changes branch February 18, 2021 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants