Storm Question

2017-11-13 Thread Otto Fowler
Are the values in a storm Tuple guaranteed to always be in the same order, such that you can reliably reference them by position and not by name? Asking for a friend. ottO

[GitHub] metron issue #824: METRON-1289: Alert fields are lost when a MetaAlert is cr...

2017-11-13 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/824 I also found what I believe are 2 fairly significant architectural issues with our DAO abstraction: - https://issues.apache.org/jira/browse/METRON-1314 -

[GitHub] metron issue #824: METRON-1289: Alert fields are lost when a MetaAlert is cr...

2017-11-13 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/824 The latest commit fixes the bug in the previous comment. The root cause was that HBaseDao.getAllLatest was not properly returning the sensor type and causing subsequent updates in Elasticsearch

[GitHub] metron issue #824: METRON-1289: Alert fields are lost when a MetaAlert is cr...

2017-11-13 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/824 I've spun this up, and `add/alert` throws this exception: ``` { "responseCode": 500, "message": "class org.apache.metron.elasticsearch.dao.ElasticsearchDao: ElasticsearchDao

[GitHub] metron pull request #826: METRON-1291: Kafka produce REST endpoint does not ...

2017-11-13 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/826#discussion_r150653926 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KafkaConfig.java --- @@ -108,6 +108,9 @@ public ZkUtils zkUtils() {

[GitHub] metron pull request #824: METRON-1289: Alert fields are lost when a MetaAler...

2017-11-13 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/824#discussion_r150644110 --- Diff: metron-platform/metron-indexing/src/test/java/org/apache/metron/indexing/dao/InMemoryMetaAlertDao.java --- @@ -200,4 +207,23 @@ public

[GitHub] metron issue #824: METRON-1289: Alert fields are lost when a MetaAlert is cr...

2017-11-13 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/824 whoops, nevermind, I see you already merged master. my bad. :) ---

[GitHub] metron pull request #824: METRON-1289: Alert fields are lost when a MetaAler...

2017-11-13 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/824#discussion_r150641377 --- Diff: metron-platform/metron-indexing/src/test/java/org/apache/metron/indexing/dao/InMemoryMetaAlertDao.java --- @@ -200,4 +207,23 @@ public

[GitHub] metron pull request #824: METRON-1289: Alert fields are lost when a MetaAler...

2017-11-13 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/824#discussion_r150628792 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/MetaAlertController.java --- @@ -60,5 +63,37 @@ ) throws

[GitHub] metron pull request #824: METRON-1289: Alert fields are lost when a MetaAler...

2017-11-13 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/824#discussion_r150638251 --- Diff: metron-platform/metron-indexing/src/test/java/org/apache/metron/indexing/dao/InMemoryMetaAlertDao.java --- @@ -200,4 +207,23 @@ public

[GitHub] metron pull request #774: METRON-1212 The bundle system and maven plugin

2017-11-13 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/774#discussion_r150620372 --- Diff: bundles-maven-plugin/README.md --- @@ -0,0 +1,230 @@ + +# Apache Metron Bundle Maven Plugin + +Apache Metron Bundles Maven

[GitHub] metron issue #824: METRON-1289: Alert fields are lost when a MetaAlert is cr...

2017-11-13 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/824 @merrimanr Can you merge master into this PR? There is at least one fix in here that caused me problems spinning this up that's in master, so I'd like to have it pulled in so testing is as stable

[GitHub] metron pull request #826: METRON-1291: Kafka produce REST endpoint does not ...

2017-11-13 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/826#discussion_r150560129 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KafkaConfig.java --- @@ -108,6 +108,9 @@ public ZkUtils zkUtils() {

[GitHub] metron pull request #774: METRON-1212 The bundle system and maven plugin

2017-11-13 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/774#discussion_r150558916 --- Diff: bundles-maven-plugin/README.md --- @@ -0,0 +1,230 @@ + +# Apache Metron Bundle Maven Plugin + +Apache Metron Bundles Maven

[GitHub] metron pull request #826: METRON-1291: Kafka produce REST endpoint does not ...

2017-11-13 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/826#discussion_r150558807 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KafkaConfig.java --- @@ -108,6 +108,9 @@ public ZkUtils zkUtils() {

[GitHub] metron pull request #826: METRON-1291: Kafka produce REST endpoint does not ...

2017-11-13 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/826#discussion_r150549974 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KafkaConfig.java --- @@ -108,6 +108,9 @@ public ZkUtils zkUtils() {