[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-08-08 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r131937974 --- Diff: bundles-lib/src/main/java/org/apache/metron/bundles/BundleClassLoaders.java --- @@ -0,0 +1,353 @@ +/* + * Licensed to the Apache

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-08-08 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r131934285 --- Diff: bundles-lib/src/main/java/org/apache/metron/bundles/BundleClassLoaders.java --- @@ -0,0 +1,353 @@ +/* + * Licensed to the Apache

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-08-08 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r131931569 --- Diff: bundles-lib/src/main/java/org/apache/metron/bundles/BundleClassLoaders.java --- @@ -0,0 +1,353 @@ +/* + * Licensed to the Apache

[GitHub] metron issue #620: Metron-988: UI for viewing alerts generated by Metron

2017-08-07 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/620 These lic. check commands needs to make their way into the build, like we do with the java stuff. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] metron pull request #626: METRON-947 [DO NOT MERGE UNTIL METRON-942] Stellar...

2017-08-07 Thread ottobackwards
Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/626 --- 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

[GitHub] metron issue #626: METRON-947 [DO NOT MERGE UNTIL METRON-942] Stellar Manage...

2017-08-07 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/626 I am closing this PR, I'll re-submit when METRON-942 is landed --- 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 pr

[GitHub] metron issue #687: METRON-1090 [NO MERGE UNTIL METRON-711] Add Assignment to...

2017-08-07 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/687 Added pre and post increment and decrement --- 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

[GitHub] metron pull request #687: METRON-1090 [NO MERGE UNTIL METRON-711] Add Assign...

2017-08-06 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/metron/pull/687 METRON-1090 [NO MERGE UNTIL METRON-711] Add Assignment to Stellar Language This PR adds support for Assignment to the Stellar Language. It is dependent on [METRON-711](https://github.com

[GitHub] metron pull request #686: METRON-711 STELLAR SHELL Do not set variables when...

2017-08-05 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/metron/pull/686 METRON-711 STELLAR SHELL Do not set variables when exceptions are thrown from assignment. This PR changes the behavior of the Stellar Shell such that when doing variable assignments ( VAR

[GitHub] metron issue #683: METRON-1084: Management UI web server license should be A...

2017-08-05 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/683 +1 --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-05 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I have changed it back to 3.0.1. I have also resolved the issue that required the metron version and archetype version to match in order to build. --- If your project is set up for it

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-04 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 @mmiklavc does that mean we can go back to 3.0.1? Or should? --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ok - so mvn archetype:generate uses the newer version of the plugin if run in an empty directory. If run in a directory where there is a pom with plugin setting for 2.4 and it will honor

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 hey @mmiklavc . Looking back at my logs above, no matter that the maven-archetype-plugin was 2.4 in the pom at the time, it shows 3.0.1 in the output. Can you check what version it shows in

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I also may have a fix for the missing catalog file issue ( or a way to run the command differently ) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 Ok, I tried with a new M2 directory and the can't find local archetype issue returned. maven-archetype-plugin

[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/682 Are the modules/versions pulled in dependent on the version of node or something? My list is different. I have more things, and different versions for some. [licenses.txt](https

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I'm going to try with a new .m2 dir --- 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 fe

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I hate to ask but are you sure you have the latest code? --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```bash ┌─[ottofowler@Winterfell] - [~/tmp/HelloParser] - [Thu Aug 03, 19:12] └─[$]> cat ~/.m2/repository/archetype-catalog.xml http://maven.apache.org/plugins/ma

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I left the Parser off of the HelloParser input, sorry --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I DO think that the archetype's input variables could use refactoring. For example, I force the artifact name, and maybe I should not. I do not know why the bundle plugin is lo

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```bash ┌─[ottofowler@Winterfell] - [~/.m2/repository/org/apache/metron/metron-maven-parser-extension-archetype/0.4.1] - [Thu Aug 03, 18:42] └─[$]> ll total 80 -rw-r

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```bash ┌─[ottofowler@Winterfell] - [~/tmp] - [Thu Aug 03, 18:36] └─[$]> mvn archetype:generate -DarchetypeCatalog=local [INFO] Scanning for projects... [I

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I'm not done the build, so I'll try this again, but this is what I get: ( Note, I don't know how you are getting the packageInPathFormat option ): ┌─[ottofo

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 WRT: the archetype directories: I don't see that ( but I'm going to re-build and try again ). I don't use the same options you are using though. Importantly ->

[GitHub] metron issue #682: METRON-1081: Fix Alerts and Ops UI Notices file

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/682 I am not sure how to verify or review this --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 Ok - I think you are running find from /usr/metron/$V/ So - to explain the other config/zookeeper So let's pretend there are two things here: 1. metron-parsers 2. p

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 This comes through better in METRON-942, where an extension built with the archetype is actually uploaded and installed into metron - ready to be activated. This is a much better flow and

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 thanks @mmiklavc. I don't think I've stated this in the actual PR As a 3rd party ( or whatever you want to call it ) developer, who only makes Parser or possibly other

[GitHub] metron pull request #681: METRON-1079 Add NaN as a keyword in STELLAR langua...

2017-08-03 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/metron/pull/681 METRON-1079 Add NaN as a keyword in STELLAR language Add "NaN" to STELLAR as a keyword. Add NaN, and tests to verify that it works in line with the java spe

[GitHub] metron pull request #679: METRON-1075: Add faceted search capabilities

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/679#discussion_r131156937 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchDao.java --- @@ -179,4 +206,43

[GitHub] metron issue #675: METRON-379 STELLAR can differentiate between a value pass...

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/675 https://issues.apache.org/jira/browse/METRON-1079 --- 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

[GitHub] metron pull request #667: METRON-1061 Add FUZZY_SCORE STELLAR function

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/667#discussion_r131155065 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/TextFunctions.java --- @@ -0,0 +1,63

[GitHub] metron issue #675: METRON-379 STELLAR can differentiate between a value pass...

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/675 I'll create a Jira for the NaN thing, unless you have already coded it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] metron pull request #675: METRON-379 STELLAR can differentiate between a val...

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/675#discussion_r131151842 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/LambdaExpression.java --- @@ -56,9 +57,9 @@ public Object

[GitHub] metron issue #675: METRON-379 STELLAR can differentiate between a value pass...

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/675 Right. This PR adds the Language Capability to required variables be passed based on the VariableResolver implementation defined. Which can be defined based on the requirements of the '

[GitHub] metron pull request #667: METRON-1061 Add FUZZY_SCORE STELLAR function

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/667#discussion_r131146245 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/TextFunctions.java --- @@ -0,0 +1,63

[GitHub] metron pull request #667: METRON-1061 Add FUZZY_SCORE STELLAR function

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/667#discussion_r131143225 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/TextFunctions.java --- @@ -0,0 +1,63

[GitHub] metron pull request #667: METRON-1061 Add FUZZY_SCORE STELLAR function

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/667#discussion_r131142516 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/TextFunctions.java --- @@ -0,0 +1,63

[GitHub] metron issue #679: METRON-1075: Add faceted search capabilities

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/679 +1 works as described. Great Job! --- 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

[GitHub] metron pull request #679: METRON-1075: Add faceted search capabilities

2017-08-03 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/679#discussion_r131128135 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchDao.java --- @@ -179,4 +206,43

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-02 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 Oh profiler integration test --- 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

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-08-02 Thread ottobackwards
GitHub user ottobackwards reopened a pull request: https://github.com/apache/metron/pull/530 METRON-777 Metron Extension System and Parser Extensions ## Contributor Comments The pr. introduces an extension system for metron, along with refactoring the metron parsers on top

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-08-02 Thread ottobackwards
Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/530 --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-08-02 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 @mmiklavc 1. I don't think we want to, or have to track it as upstream. But we do want to keep track of bug fixes and improvements to areas where we have not deviated. That

[GitHub] metron issue #676: METRON-1072: Cleanup Maven build warnings

2017-08-02 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/676 +1, built with full commit build options, ran full dev, saw data. Honestly, these errors are not what flood my logs, but it is good to chip away --- If your project is set up for it, you

[GitHub] metron issue #662: METRON-1056: Get field types from Elasticsearch

2017-08-01 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/662 +1 --- 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

[GitHub] metron pull request #675: METRON-379 STELLAR can differentiate between a val...

2017-08-01 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/metron/pull/675 METRON-379 STELLAR can differentiate between a value passed as null and a value not passed That being that there IS a variable at all vs. there is a variable, and its value is NULL

[GitHub] metron issue #674: METRON-681 CSVConverter should trim key and values

2017-08-01 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/674 @basvdl please be sure to take care of the jira --- 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

[GitHub] metron issue #674: METRON-681 CSVConverter should trim key and values

2017-07-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/674 Yes. Does X with incoming .. trims all values --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-30 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 "Based on feedback from reviews and discussion with mattf, this commit… … represent a major refactoring for the bundle functionality. While trying to resolve the ugliness b

[GitHub] metron issue #674: METRON-681 CSVConverter should trim key and values

2017-07-29 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/674 Thanks for the PR. I think that we should document the behavior somehow. Can you add a general class javadoc? --- If your project is set up for it, you can reply to this email and have your

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-28 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r130213711 --- Diff: bundles-lib/src/main/java/org/apache/metron/bundles/util/VFSClassloaderUtil.java --- @@ -0,0 +1,178 @@ +/** + * Licensed to the

[GitHub] metron issue #665: METRON-1060: Add performance timing logging to enrichment...

2017-07-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/665 +1 -> Love this. Really great work. Makes me think that we want to have some kind of topical area of the site-book or developers documentation ( like this would go with testing ) but that

[GitHub] metron issue #665: METRON-1060: Add performance timing logging to enrichment...

2017-07-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/665 Nice, tag me when it is in --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 METRON-1070 --- 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

[GitHub] metron issue #607: METRON-982 add new rest api for storm supervisor status

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/607 @merrimanr we all set? --- 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

[GitHub] metron issue #667: METRON-1061 Add FUZZY_SCORE STELLAR function

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/667 @JonZeolla --- 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

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 @mattf-horton I have reviewed the changes between what I have and 1.3.0, and while there are some one or two slight differences, there is nothing there that functionally improves or applies to

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 @mattf-horton I think I have address the issues up until now. I have not done the compare to nifi 1.3.0 to see the difference however. --- If your project is set up for it, you can reply to

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```bash . ├── java │   └── org │   └── apache │   └── metron │   ├── bundles │ Â

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 No, there is no problem with you referring back to nifi @simonellistonball --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 That CEFParserTest is actually from someone else's 'appropriation' of Nifi code ;) --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 WRT: Required Bundle dependency for BundleProvidedDependenciesMojo. That mojo is not run automatically, either in metron or in the nifi case. It is *explicitly* run to determine the

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129946569 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 I'm actually going to debug this and verify about the dependency --- 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 pr

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 > My question was, since it throws an exception if no Bundle dependency is stated, if I have a Bundle that is not actually dependent on any other bundles, what should I declare it is depend

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129933909 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129933315 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129932703 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129929213 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129923353 --- Diff: metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java --- @@ -182,6 +185,16

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129920992 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/SupervisorStatus.java --- @@ -0,0 +1,127

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129920967 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/SupervisorStatus.java --- @@ -0,0 +1,127

[GitHub] metron pull request #607: METRON-982 add new rest api for storm supervisor s...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/607#discussion_r129908796 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/StormStatusServiceImpl.java --- @@ -17,10 +17,7

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827680 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827745 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827293 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827277 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827115 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129827184 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleProvidedDependenciesMojo.java --- @@ -0,0 +1,326 @@ +/* + * Licensed to

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826663 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826699 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826628 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826587 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826645 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826679 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826513 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826554 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826535 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826467 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129826380 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129825816 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129825779 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129824553 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron pull request #530: METRON-777 Metron Extension System and Parser Exte...

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/530#discussion_r129824590 --- Diff: bundles-maven-plugin/src/main/java/org/apache/metron/BundleMojo.java --- @@ -0,0 +1,742 @@ +/* + * Licensed to the Apache Software

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 WRT: Maven Plugin versions : There are no changes that are not my changes, so we have the 1.2.0 release code --- If your project is set up for it, you can reply to this email and have your

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-27 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 ```code - To have a single lib, some things that were in other nifi libraries had to be relocated. ./bundles: ExtensionManager.java-> MODIF

[GitHub] metron issue #530: METRON-777 Metron Extension System and Parser Extensions

2017-07-26 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/530 Thanks for the feedback @mattf-horton, I will go through it all. About tracking changes... I will check and see what there is to pick up. Obviously this was much more current when the PR was

<    5   6   7   8   9   10   11   12   13   14   >