[GitHub] metron issue #969: METRON-1496: ChainLink Parser to reuse parser code at par...

2018-03-28 Thread kevin91nl
Github user kevin91nl commented on the issue: https://github.com/apache/metron/pull/969 What exact changes are needed such that it fits in the existing architecture? ---

[GitHub] metron issue #969: METRON-1496: ChainLink Parser to reuse parser code at par...

2018-03-21 Thread kevin91nl
Github user kevin91nl commented on the issue: https://github.com/apache/metron/pull/969 @nickwallen We used Jinjava for rendering Jinja templates, but that drastically decreased the EPS. Now we use simple replaces of variables, but it is less flexible than the template render engine

[GitHub] metron issue #969: METRON-1496: ChainLink Parser to reuse parser code at par...

2018-03-21 Thread kevin91nl
Github user kevin91nl commented on the issue: https://github.com/apache/metron/pull/969 The package naming is adjusted in the latest commit. **Chainparsers versus Stellar** Here I will present some of the issues we encountered when writing the parsers. I am interested in

[GitHub] metron issue #969: METRON-1496: ChainLink Parser to reuse parser code at par...

2018-03-20 Thread kevin91nl
Github user kevin91nl commented on the issue: https://github.com/apache/metron/pull/969 We would like to share our configurable parser architecture for review and discussion. Our parsers are currently found in `metron-platform/metron-parsers-contrib` which might be changed. ---

[GitHub] metron pull request #969: METRON-1496: ChainLink Parser to reuse parser code...

2018-03-20 Thread kevin91nl
GitHub user kevin91nl opened a pull request: https://github.com/apache/metron/pull/969 METRON-1496: ChainLink Parser to reuse parser code at parserConfig level ## Contributor Comments [Please place any comments here. A description of the problem/enhancement, how to reproduce