Deep recursion postfix patterns#69
Merged
anthonycorbacho merged 2 commits intothekrakken:masterfrom Aug 4, 2017
Merged
Conversation
Contributor
Author
|
Also completes the PR #29, solves the same problem. |
retoo
force-pushed
the
deep-recursion-postfix-patterns
branch
from
1fc8bd7 to
59f711e
Compare
Contributor
Author
|
@anthonycorbacho I already resolved the conflict this PR, please merge the Pull Request #70 first, afterwards you can merge this one (which then should have no conflict). |
Member
The Postfix grok patterns have been downloaded from https://github.com/whyscream/postfix-grok-patterns/blob/f0ec34dcc6250463a30ba2077d8afa89ee1a17a1/postfix.grok Refs Graylog2/graylog2-server#3949
retoo
force-pushed
the
deep-recursion-postfix-patterns
branch
from
59f711e to
e4c9547
Compare
Contributor
Author
|
@anthonycorbacho strange; no idea why it didn't work. Anyway, now it's rebased. |
Member
|
Awesome thanks !! |
Contributor
|
@anthonycorbacho Would it be possible to get a release of java-grok with this fix? |
Contributor
Author
|
@anthonycorbacho yes please! @joschi you can use the jetpack.io distribution if you like: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Incorporates the tests from #67 and fixes #66
Grok/compile aborts now if a definition cannot be resolved.
@anthonycorbacho I have no idea often this might break actual setups. The unit tests are green ;)