GitHub user danieljue opened a pull request:

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

    Metron 451

    Fix for METRON-451
    SerDeUtils - java.lang.ClassNotFoundException: 
org.apache.storm.shade.org.joda.time.DateTime
    
    Sorry guys and gals, forgot to put this fix in it's own branch, might have 
screwed up previous PR.

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

    $ git pull https://github.com/Cognami/incubator-metron METRON-451

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

    https://github.com/apache/incubator-metron/pull/278.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 #278
    
----
commit 6d54f7b09427a1859d073303e22ace64e42e21b3
Author: Daniel Jue <[email protected]>
Date:   2016-09-26T16:06:39Z

    Fixed package error.

commit 1e45d059067f50afeadc22da13e3320990585367
Author: Daniel Jue <[email protected]>
Date:   2016-09-27T14:07:47Z

    Fix for METRON-451
    SerDeUtils - java.lang.ClassNotFoundException: 
org.apache.storm.shade.org.joda.time.DateTime
    
    Took the same jodatime dependency from API's pom and put it in Common's 
pom.  Not sure how this was building otherwise--it should not have been 
build-able without this dependency visible to this module.
    
    Suggest adding the version number as a variable that is controlled at a 
parent pom's dependency management section, so that the version numbers don't 
get out of sync between modules.

commit 8f83fb439392ef221497dae1abdd7b5309102be2
Author: Daniel Jue <[email protected]>
Date:   2016-09-27T14:09:03Z

    Merge branch 'master' of https://github.com/apache/incubator-metron
    
    # By cstella (4) and others
    # Via cstella
    * 'master' of https://github.com/apache/incubator-metron:
      METRON-457 Correct GrokParser logging spelling error (mmiklavc via 
cestella) closes apache/incubator-metron#274
      METRON-449 JSONMapParser should unfold maps to arbitrary depths closes 
apache/incubator-metron#271
      METRON-453: Add a stellar shell function to open an external editor and 
return the editor's contents closes apache/incubator-metron#272
      METRON-452: Add rudimentary configuration management functions to Stellar 
closes apache/incubator-metron#269
      METRON-374: Add appropriate bundled 3rd party licenses to NOTICE and 
LICENSE where appropriate closes apache/incubator-metron#229
      flush
      METRON-427 Create Ambari Management Pack for Metron Installation closes 
apache/incubator-metron#266

commit 896dc51ca0989170e91be709ecd40d020aeb35b9
Author: Daniel Jue <[email protected]>
Date:   2016-09-27T14:12:17Z

    Revert "Fix for METRON-451"
    
    This reverts commit 1e45d059067f50afeadc22da13e3320990585367.

commit 75162567e6890f76f30d0b555724212045276afc
Author: Daniel Jue <[email protected]>
Date:   2016-09-27T14:07:47Z

    Fix for METRON-451
    SerDeUtils - java.lang.ClassNotFoundException: 
org.apache.storm.shade.org.joda.time.DateTime
    
    Took the same jodatime dependency from API's pom and put it in Common's 
pom.  Not sure how this was building otherwise--it should not have been 
build-able without this dependency visible to this module.
    
    Suggest adding the version number as a variable that is controlled at a 
parent pom's dependency management section, so that the version numbers don't 
get out of sync between modules.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to