[GitHub] incubator-metron pull request #459: METRON-726: Clean up mvn site generation

2017-02-19 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/incubator-metron/pull/459 METRON-726: Clean up mvn site generation ### Summary An initial attempt to add checkstyle expanded a bit once it turned out that `mvn site` appears to not work right now (and

[GitHub] incubator-metron pull request #458: METRON-725: Javadoc is broken by the use...

2017-02-19 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/incubator-metron/pull/458 METRON-725: Javadoc is broken by the use of apiNote As noted in the ticket, there are 3 options for fixing this 1. Just delete the offending inherited javadoc 2. Explicitly

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101933431 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/GrokController.java --- @@ -0,0 +1,56 @@ +/** + *

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101934020 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/StellarController.java --- @@ -0,0 +1,80 @@ +/**

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101934168 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/SensorEnrichmentConfigController.java --- @@ -0,0 +1,97

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101933196 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/HdfsController.java --- @@ -0,0 +1,89 @@ +/** + *

Re: [DISCUSS][PROPOSAL] Side Loading and Installation of telemetry sources [METRON-258]

2017-02-19 Thread zeo...@gmail.com
Awesome write up and ideas Otto, I also strongly support this idea. As someone who has the development of a few parsers quickly approaching the top of their to do list, I will happily beta test this for you when it's far enough along for that. Until then I will attempt to take a look at your

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101921984 --- Diff: metron-interface/metron-rest/pom.xml --- @@ -0,0 +1,335 @@ + + + +http://maven.apache.org/POM/4.0.0;

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101922012 --- Diff: metron-interface/pom.xml --- @@ -0,0 +1,90 @@ + + + +http://maven.apache.org/POM/4.0.0;

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101922040 --- Diff: metron-interface/metron-rest/src/test/resources/README.vm --- @@ -0,0 +1,142 @@ +#[[#]]# Metron REST + +This module provides

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-02-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r101921992 --- Diff: metron-interface/metron-rest/src/main/resources/application.yml --- @@ -0,0 +1,42 @@ +# Licensed to the Apache Software Foundation