You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@metron.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/09/24 12:56:20 UTC

[jira] [Commented] (METRON-449) JSON Parser should handle arbitrarily deep nesting of maps

    [ https://issues.apache.org/jira/browse/METRON-449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15519009#comment-15519009 ] 

ASF GitHub Bot commented on METRON-449:
---------------------------------------

GitHub user ottobackwards opened a pull request:

    https://github.com/apache/incubator-metron/pull/270

    METRON-449 - JSON Parser should unfold nested maps

    For greater flexibility and utility the JSONMapParser should support unfolding maps to arbitrary depths.
    
    This change introduces a new function called recursively to handle this, because the enum lambdas cannot reference themselves.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ottobackwards/incubator-metron METRON-449

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-metron/pull/270.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #270
    
----
commit 0c8bb5c4ab04f9e19859981cea5bfe58b868a16f
Author: cstella <ce...@gmail.com>
Date:   2016-09-07T20:47:03Z

    Initial commit for identity parser.

commit 151a54b07832aef82c95a4fd59a2b0a0bc30e005
Author: cstella <ce...@gmail.com>
Date:   2016-09-19T13:07:28Z

    Merge branch 'master' into identity_parser

commit ffa7b5297f4013aa7aa364f49bae1c2eb6c4ee3a
Author: cstella <ce...@gmail.com>
Date:   2016-09-19T15:13:07Z

    updating to have a handler for maps.

commit 68be22f5e1b72172e415909340c121c974651414
Author: cstella <ce...@gmail.com>
Date:   2016-09-19T22:14:04Z

    METRON-434: JSON Parser

commit 860e8302cb1edf0c9178b37093a7930cd068ff92
Author: cstella <ce...@gmail.com>
Date:   2016-09-19T22:17:49Z

    Merge branch 'identity_parser' into METRON-434

commit 3adca211b7836af9b559be72125c2fdab3ac7b44
Author: cstella <ce...@gmail.com>
Date:   2016-09-20T13:17:00Z

    licenses

commit 8a725e99fb23bf12973a0778e54cd70b927951e2
Author: cstella <ce...@gmail.com>
Date:   2016-09-20T13:39:26Z

    Missed a license.

commit b8af693a85aa5b2ef60e08a528eeb92741a86432
Author: cstella <ce...@gmail.com>
Date:   2016-09-20T15:03:07Z

    Updating test.

commit 3a986c4361fd1755b75089133600f0e26e9cc34c
Author: Otto Fowler <of...@industrialdefender.com>
Date:   2016-09-21T19:55:45Z

    Merge branch METRON-434 of https://github.com/cestella/incubator-metron into METRON-434

commit 2f17d5938865b6a989247020df90921d79ab6f74
Author: Otto Fowler <of...@industrialdefender.com>
Date:   2016-09-21T20:36:54Z

    JSONMapParser should handle unfolding nested maps
    
    The JSONMapParser only supports unfolding of maps at the root of the
    JSON object.  It would be better if it could unfold to arbitrary depths
    for broader range of support.
    
    This is added by introducing recursion to the UNFOLD functionality.
    Because you can’t self reference in the enum lambda, I have added a
    private function which can recurse and had the lambda call that.
    It is possible that there is a better way to implement this, so
    hopefully that will come out in review.

commit 6f1025f7729540f943e8df8d99f282ca98b96f57
Author: Otto Fowler <of...@industrialdefender.com>
Date:   2016-09-24T12:21:47Z

    Merge remote-tracking branch 'apache/master'

commit c8b949554d64cc45d9298a0bead04315e0a9ea10
Author: Otto Fowler <of...@industrialdefender.com>
Date:   2016-09-24T12:41:40Z

    merge from apache master

----


> JSON Parser should handle arbitrarily deep nesting of maps
> ----------------------------------------------------------
>
>                 Key: METRON-449
>                 URL: https://issues.apache.org/jira/browse/METRON-449
>             Project: Metron
>          Issue Type: Improvement
>    Affects Versions: 0.2.1BETA
>            Reporter: Otto Fowler
>
> The JSONMapParser's unfolding function only unfolds maps at the root level
> such as "collection" : { "blah" : 7, "blah2" : "foo" }.
> This limits the utility of the parser, since it is likely that someone would have to do a good deal of transformation before hand.
> The ability for the parser to handle nested objects ( maps ) to arbitrary depth would greatly add to the utility of this parser.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)