search

thekrakken / java-grok

Simple API that allows you to easily parse logs and other files
http://grok.nflabs.com/
Other
358 stars 152 forks source link

Deep recursion postfix patterns #69

Closed retoo closed 7 years ago

retoo commented 7 years ago

Incorporates the tests from https://github.com/thekrakken/java-grok/pull/67 and fixes https://github.com/thekrakken/java-grok/issues/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 ;)

retoo commented 7 years ago

Also completes the PR https://github.com/thekrakken/java-grok/pull/29, solves the same problem.

retoo commented 7 years ago

@anthonycorbacho I already resolved the conflict this PR, please merge the Pull Request https://github.com/thekrakken/java-grok/pull/70 first, afterwards you can merge this one (which then should have no conflict).

anthonycorbacho commented 7 years ago

@retoo I merged #70

Once you resolve the conflict here, i will merge this pr as well :)

retoo commented 7 years ago

@anthonycorbacho strange; no idea why it didn't work. Anyway, now it's rebased.

anthonycorbacho commented 7 years ago

Awesome thanks !!

joschi commented 6 years ago

@anthonycorbacho Would it be possible to get a release of java-grok with this fix?

retoo commented 6 years ago

@anthonycorbacho yes please!

@joschi you can use the jetpack.io distribution if you like: => https://jitpack.io/#thekrakken/java-grok/551fadba59