[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311169680 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311169680 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311169680 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311165058 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/V

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311156658 ## File path: metron-analytics/metron-profiler-storm/src/test/java/org/apache/metron/profiler/storm/integration

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311154623 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/G

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311145031 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6 +6

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311143515 ## File path: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/hbase/SaltyRowKeyB

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311141511 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311137190 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311134907 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311132873 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311131248 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/ProfilerCli

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311129722 ## File path: metron-analytics/metron-profiler-storm/src/test/java/org/apache/metron/profiler/storm/integration/P

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311098015 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/V

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311083234 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/G

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311075260 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/G

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311081338 ## File path: dependencies_with_url.csv ## @@ -146,6 +146,7 @@ com.codahale.metrics:metrics-core:jar:

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311063115 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/stellar/G

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311072877 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311072877 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311058843 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/ProfilerC

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311075260 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/G

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311072877 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311064624 ## File path: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/hbase/SaltyRowKe

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311063115 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/stellar/G

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311058843 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/ProfilerC

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311057827 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/HBaseProf

[GitHub] [metron] nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-08-06 Thread GitBox
nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r311053614 ## File path: metron-analytics/metron-profiler-storm/src/test/java/org/apache/metron/profiler/storm/integration

[GitHub] [metron] sardell opened a new pull request #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-06 Thread GitBox
sardell opened a new pull request #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464 ## Contributor Comments Link to original JIRA ticket here: https://issues.apache.org/jira/browse/METRON-2179 A PR was already opened for th

[GitHub] [metron] sardell closed pull request #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-06 Thread GitBox
sardell closed pull request #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [metron] asfgit closed pull request #1473: METRON-2195 Add defensive log level checks when constructing logs is expensive

2019-08-05 Thread GitBox
asfgit closed pull request #1473: METRON-2195 Add defensive log level checks when constructing logs is expensive URL: https://github.com/apache/metron/pull/1473 This is an automated message from the Apache Git Service. To re

[GitHub] [metron] mmiklavc commented on issue #1473: METRON-2195 Add defensive log level checks when constructing logs is expensive

2019-08-05 Thread GitBox
mmiklavc commented on issue #1473: METRON-2195 Add defensive log level checks when constructing logs is expensive URL: https://github.com/apache/metron/pull/1473#issuecomment-518268593 +1 This is an automated message from the

[GitHub] [metron] asfgit closed pull request #1476: METRON-2202: Add parameter validation for the stellar field validation functions.

2019-08-05 Thread GitBox
asfgit closed pull request #1476: METRON-2202: Add parameter validation for the stellar field validation functions. URL: https://github.com/apache/metron/pull/1476 This is an automated message from the Apache Git Service. To

[GitHub] [metron] asfgit closed pull request #1475: METRON-2197 Add debugging info output for Solr queries

2019-08-05 Thread GitBox
asfgit closed pull request #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [metron] ottobackwards commented on issue #1475: METRON-2197 Add debugging info output for Solr queries

2019-08-04 Thread GitBox
ottobackwards commented on issue #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475#issuecomment-518026447 +1, this looks great. Thanks! This is an automated message

[GitHub] [metron] ottobackwards commented on a change in pull request #1475: METRON-2197 Add debugging info output for Solr queries

2019-08-04 Thread GitBox
ottobackwards commented on a change in pull request #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475#discussion_r310398924 ## File path: metron-platform/metron-solr/metron-solr-common/src/main/java/org/apache/metron/solr/

[GitHub] [metron] mmiklavc commented on issue #1341: METRON-614: Eliminate use of the default Charset

2019-08-02 Thread GitBox
mmiklavc commented on issue #1341: METRON-614: Eliminate use of the default Charset URL: https://github.com/apache/metron/pull/1341#issuecomment-517735194 fyi, I'm also seeing this failing unit test `shouldPopulateMessagesOnProcessMessage`. Looking into it - it's probably some flakiness w

[GitHub] [metron] ruffle1986 closed pull request #1427: METRON-2136: Add parser aggregation sidebar

2019-08-02 Thread GitBox
ruffle1986 closed pull request #1427: METRON-2136: Add parser aggregation sidebar URL: https://github.com/apache/metron/pull/1427 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [metron] sardell commented on issue #1427: METRON-2136: Add parser aggregation sidebar

2019-08-02 Thread GitBox
sardell commented on issue #1427: METRON-2136: Add parser aggregation sidebar URL: https://github.com/apache/metron/pull/1427#issuecomment-517683233 +1 This is an automated message from the Apache Git Service. To respond to th

[GitHub] [metron] sardell edited a comment on issue #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-02 Thread GitBox
sardell edited a comment on issue #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464#issuecomment-517651590 @merrimanr I closed and reopened, but it looks like we the build is still timing out. It's odd because it looks to me like the

[GitHub] [metron] sardell commented on issue #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-02 Thread GitBox
sardell commented on issue #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464#issuecomment-517651590 @merrimanr I closed and reopened, but it looks like we are still timing out. It's odd because it looks to me like the build is succes

[GitHub] [metron] sardell opened a new pull request #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-01 Thread GitBox
sardell opened a new pull request #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464 ## Contributor Comments Link to original JIRA ticket here: https://issues.apache.org/jira/browse/METRON-2179 A PR was already opened for th

[GitHub] [metron] sardell closed pull request #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-01 Thread GitBox
sardell closed pull request #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [metron] asfgit closed pull request #1467: METRON-2189 Optimize imports in mpack python scripts

2019-08-01 Thread GitBox
asfgit closed pull request #1467: METRON-2189 Optimize imports in mpack python scripts URL: https://github.com/apache/metron/pull/1467 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [metron] mmiklavc commented on issue #1467: METRON-2189 Optimize imports in mpack python scripts

2019-08-01 Thread GitBox
mmiklavc commented on issue #1467: METRON-2189 Optimize imports in mpack python scripts URL: https://github.com/apache/metron/pull/1467#issuecomment-517431113 > Assuming you ran this up in full dev (happy to do it if not), +1 by inspection. Yes, I ran this up to make sure I didn't b

[GitHub] [metron] mmiklavc commented on issue #1473: METRON-2195 rebase onto master

2019-08-01 Thread GitBox
mmiklavc commented on issue #1473: METRON-2195 rebase onto master URL: https://github.com/apache/metron/pull/1473#issuecomment-517430823 I missed one thing, which was requested by @ottobackwards on the Jira ticket - can you port my perf test into this PR? Don't have it run as part of a norm

[GitHub] [metron] merrimanr commented on issue #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-01 Thread GitBox
merrimanr commented on issue #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464#issuecomment-517404217 Looks like the build timed out. I noticed this same issue in the PR I submitted against this. Could it be specific to this PR? @

[GitHub] [metron] merrimanr commented on issue #1467: METRON-2189 Optimize imports in mpack python scripts

2019-08-01 Thread GitBox
merrimanr commented on issue #1467: METRON-2189 Optimize imports in mpack python scripts URL: https://github.com/apache/metron/pull/1467#issuecomment-517402444 Assuming you ran this up in full dev (happy to do it if not), +1 by inspection. -

[GitHub] [metron] mmiklavc commented on issue #1473: METRON-2195 rebase onto master

2019-08-01 Thread GitBox
mmiklavc commented on issue #1473: METRON-2195 rebase onto master URL: https://github.com/apache/metron/pull/1473#issuecomment-517389537 One ask - can you update your PR title to match the Jira description? Thanks! This is an

[GitHub] [metron] mmiklavc commented on a change in pull request #1473: METRON-2195 rebase onto master

2019-08-01 Thread GitBox
mmiklavc commented on a change in pull request #1473: METRON-2195 rebase onto master URL: https://github.com/apache/metron/pull/1473#discussion_r309816959 ## File path: metron-platform/metron-common/src/test/java/org/apache/metron/common/utils/LazyLoggerImplTest.java ## @

[GitHub] [metron] mmiklavc edited a comment on issue #1475: METRON-2197 Add debugging info output for Solr queries

2019-08-01 Thread GitBox
mmiklavc edited a comment on issue #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475#issuecomment-517376520 @ottobackwards I tinkered with a reformat. Here's what it looks like if I merge them into a single debug statement

[GitHub] [metron] mmiklavc commented on issue #1475: METRON-2197 Add debugging info output for Solr queries

2019-08-01 Thread GitBox
mmiklavc commented on issue #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475#issuecomment-517376520 @ottobackwards I tinkered with a reformat. Here's what it looks like if I merge them into a single debug statement ```

[GitHub] [metron] mmiklavc commented on issue #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-01 Thread GitBox
mmiklavc commented on issue #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464#issuecomment-517371990 Nice work guys! I'm +1 pending a successful Travis build. This is a

[GitHub] [metron] mmiklavc commented on issue #1476: METRON-2202: Add parameter validation for the stellar field validation functions.

2019-08-01 Thread GitBox
mmiklavc commented on issue #1476: METRON-2202: Add parameter validation for the stellar field validation functions. URL: https://github.com/apache/metron/pull/1476#issuecomment-517371059 I'll pile on with my +1 as well, nice work @MohanDV and thanks for the submission! --

[GitHub] [metron] mmiklavc commented on a change in pull request #1475: METRON-2197 Add debugging info output for Solr queries

2019-08-01 Thread GitBox
mmiklavc commented on a change in pull request #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475#discussion_r309764287 ## File path: metron-platform/metron-solr/metron-solr-common/src/main/java/org/apache/metron/solr/dao/S

[GitHub] [metron] ruffle1986 closed pull request #1426: METRON-2135: Add NgRx actions to trigger state changes

2019-08-01 Thread GitBox
ruffle1986 closed pull request #1426: METRON-2135: Add NgRx actions to trigger state changes URL: https://github.com/apache/metron/pull/1426 This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [metron] ottobackwards commented on issue #1476: METRON-2202: Add parameter validation for the stellar field validation functions.

2019-08-01 Thread GitBox
ottobackwards commented on issue #1476: METRON-2202: Add parameter validation for the stellar field validation functions. URL: https://github.com/apache/metron/pull/1476#issuecomment-517300192 +1 by inspection, thanks! This

[GitHub] [metron] sardell commented on issue #1426: METRON-2135: Add NgRx actions to trigger state changes

2019-08-01 Thread GitBox
sardell commented on issue #1426: METRON-2135: Add NgRx actions to trigger state changes URL: https://github.com/apache/metron/pull/1426#issuecomment-517215930 +1 This is an automated message from the Apache Git Service. To r

[GitHub] [metron] sardell commented on issue #1464: METRON-2179: Make navigation in both UIs consistent

2019-08-01 Thread GitBox
sardell commented on issue #1464: METRON-2179: Make navigation in both UIs consistent URL: https://github.com/apache/metron/pull/1464#issuecomment-517209258 Alright, @mmiklavc. I think we are now covered for customized ports thanks to @merrimanr's contribution. Thank you both for helping m

[GitHub] [metron] MohanDV opened a new pull request #1476: METRON-2202: Add parameter validation for the stellar field validation functions.

2019-08-01 Thread GitBox
MohanDV opened a new pull request #1476: METRON-2202: Add parameter validation for the stellar field validation functions. URL: https://github.com/apache/metron/pull/1476 ## Contributor Comments Some of the field validation functions in the stellar returns true for empty parameters ,

[GitHub] [metron] ottobackwards commented on a change in pull request #1475: METRON-2197 Add debugging info output for Solr queries

2019-07-31 Thread GitBox
ottobackwards commented on a change in pull request #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475#discussion_r309407506 ## File path: metron-platform/metron-solr/metron-solr-common/src/main/java/org/apache/metron/solr/

[GitHub] [metron] mmiklavc opened a new pull request #1475: METRON-2197 Add debugging info output for Solr queries

2019-07-31 Thread GitBox
mmiklavc opened a new pull request #1475: METRON-2197 Add debugging info output for Solr queries URL: https://github.com/apache/metron/pull/1475 ## Contributor Comments https://issues.apache.org/jira/browse/METRON-2197 ## Pull Request Checklist Thank you for submitting

[GitHub] [metron] ruffle1986 closed pull request #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store

2019-07-31 Thread GitBox
ruffle1986 closed pull request #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store URL: https://github.com/apache/metron/pull/1425 This is an automated message from the Apache Git Service.

[GitHub] [metron] MohanDV opened a new pull request #1474: METRON-2201: The description for the IS_IP method default behavior needs to corrected as per implementation

2019-07-31 Thread GitBox
MohanDV opened a new pull request #1474: METRON-2201: The description for the IS_IP method default behavior needs to corrected as per implementation URL: https://github.com/apache/metron/pull/1474 ## Contributor Comments The description for the IS_IP method default behavior needs to corr

[GitHub] [metron] tigerquoll commented on issue #1473: METRON-2195 rebase onto master

2019-07-31 Thread GitBox
tigerquoll commented on issue #1473: METRON-2195 rebase onto master URL: https://github.com/apache/metron/pull/1473#issuecomment-516816901 Since this is supposed to be a temporary fix pending Metron moving to SLF4J version 2.x, I've tried to minimise changes to those areas that would get a

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308922184 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308923711 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r309002330 ## File path: metron-analytics/metron-profiler-storm/src/main/config/profiler.properties ## @@ -61,6 +6

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r309003649 ## File path: metron-analytics/metron-profiler-storm/src/test/java/org/apache/metron/profiler/storm/integration/P

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r309002603 ## File path: metron-analytics/metron-profiler-storm/src/main/java/org/apache/metron/hbase/bolt/HBaseBolt.java #

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308915461 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/HBaseProfil

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308898766 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/HBaseProfil

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308921540 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/ProfileWrit

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308910926 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Ver

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308999830 ## File path: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/hbase/SaltyRowKeyB

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308921042 ## File path: metron-analytics/metron-profiler-client/src/test/java/org/apache/metron/profiler/client/ProfileGene

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308897952 ## File path: dependencies_with_url.csv ## @@ -146,6 +146,7 @@ com.codahale.metrics:metrics-core:jar:3.

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r309001264 ## File path: metron-analytics/metron-profiler-spark/src/main/java/org/apache/metron/profiler/spark/function/HBas

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r309002912 ## File path: metron-analytics/metron-profiler-storm/src/test/java/org/apache/metron/hbase/bolt/Widget.java

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r309001816 ## File path: metron-analytics/metron-profiler-spark/src/test/java/org/apache/metron/profiler/spark/function/HBas

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308904532 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308909674 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Ver

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308902696 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/ProfilerCli

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308907036 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Ver

[GitHub] [metron] mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-30 Thread GitBox
mmiklavc commented on a change in pull request #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#discussion_r308906231 ## File path: metron-analytics/metron-profiler-client/src/main/java/org/apache/metron/profiler/client/stellar/Get

[GitHub] [metron] merrimanr commented on issue #1436: METRON-2149: Shaded jar classifier is not consistent

2019-07-30 Thread GitBox
merrimanr commented on issue #1436: METRON-2149: Shaded jar classifier is not consistent URL: https://github.com/apache/metron/pull/1436#issuecomment-516579063 Due the the extensive testing that will be required for this, I am planning on closing and reopening against the METRON-2088-suppo

[GitHub] [metron] merrimanr closed pull request #1436: METRON-2149: Shaded jar classifier is not consistent

2019-07-30 Thread GitBox
merrimanr closed pull request #1436: METRON-2149: Shaded jar classifier is not consistent URL: https://github.com/apache/metron/pull/1436 This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [metron] merrimanr commented on issue #1462: METRON-2181: SolrDao should include aliases

2019-07-30 Thread GitBox
merrimanr commented on issue #1462: METRON-2181: SolrDao should include aliases URL: https://github.com/apache/metron/pull/1462#issuecomment-516578530 I agree, wildcards is probably a better option here. Will open a separate PR for that. ---

[GitHub] [metron] merrimanr closed pull request #1462: METRON-2181: SolrDao should include aliases

2019-07-30 Thread GitBox
merrimanr closed pull request #1462: METRON-2181: SolrDao should include aliases URL: https://github.com/apache/metron/pull/1462 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [metron] sardell commented on issue #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store

2019-07-30 Thread GitBox
sardell commented on issue #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store URL: https://github.com/apache/metron/pull/1425#issuecomment-516484776 +1 This is an automated message from the

[GitHub] [metron] tigerquoll commented on a change in pull request #1473: METRON-2195 rebase onto master

2019-07-30 Thread GitBox
tigerquoll commented on a change in pull request #1473: METRON-2195 rebase onto master URL: https://github.com/apache/metron/pull/1473#discussion_r308709556 ## File path: metron-platform/metron-common/src/main/java/org/apache/metron/common/utils/LazyLogger.java ## @@ -0,0

[GitHub] [metron] ruffle1986 commented on a change in pull request #1430: METRON-2139: Refactoring sensor-parser-config.component and wire NgRx

2019-07-30 Thread GitBox
ruffle1986 commented on a change in pull request #1430: METRON-2139: Refactoring sensor-parser-config.component and wire NgRx URL: https://github.com/apache/metron/pull/1430#discussion_r308685706 ## File path: metron-interface/metron-config/src/app/sensors/reducers/sensors.reducers

[GitHub] [metron] sardell commented on a change in pull request #1430: METRON-2139: Refactoring sensor-parser-config.component and wire NgRx

2019-07-30 Thread GitBox
sardell commented on a change in pull request #1430: METRON-2139: Refactoring sensor-parser-config.component and wire NgRx URL: https://github.com/apache/metron/pull/1430#discussion_r308683716 ## File path: metron-interface/metron-config/src/app/sensors/reducers/sensors.reducers.ts

[GitHub] [metron] sardell commented on a change in pull request #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store

2019-07-30 Thread GitBox
sardell commented on a change in pull request #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store URL: https://github.com/apache/metron/pull/1425#discussion_r308679646 ## File path: metron-interface/metron-config/src/app/sensors/reducers/sensors.r

[GitHub] [metron] sardell commented on a change in pull request #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store

2019-07-30 Thread GitBox
sardell commented on a change in pull request #1425: METRON-2134: Add NgRx reducers to perform parser and group changes in the store URL: https://github.com/apache/metron/pull/1425#discussion_r308679646 ## File path: metron-interface/metron-config/src/app/sensors/reducers/sensors.r

[GitHub] [metron] ruffle1986 closed pull request #1424: METRON-2133: Add NgRx effects to communicate with the server

2019-07-29 Thread GitBox
ruffle1986 closed pull request #1424: METRON-2133: Add NgRx effects to communicate with the server URL: https://github.com/apache/metron/pull/1424 This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [metron] merrimanr commented on issue #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2

2019-07-29 Thread GitBox
merrimanr commented on issue #1458: METRON-2177 Upgrade Profiler for HBase 2.0.2 URL: https://github.com/apache/metron/pull/1458#issuecomment-516178822 I ran through the testing instructions and hit some problems with the Batch Profiler. First error I ran into was this: ``` 19/07/29

[GitHub] [metron] mmiklavc commented on a change in pull request #1473: METRON-2195 rebase onto master

2019-07-29 Thread GitBox
mmiklavc commented on a change in pull request #1473: METRON-2195 rebase onto master URL: https://github.com/apache/metron/pull/1473#discussion_r308428770 ## File path: metron-platform/metron-common/src/main/java/org/apache/metron/common/utils/LazyLogger.java ## @@ -0,0 +

[GitHub] [metron] asfgit closed pull request #1471: METRON-2194 Update Ambari tooltip to specify single quotes for parser names with hyphens

2019-07-29 Thread GitBox
asfgit closed pull request #1471: METRON-2194 Update Ambari tooltip to specify single quotes for parser names with hyphens URL: https://github.com/apache/metron/pull/1471 This is an automated message from the Apache Git Serv

<    9   10   11   12   13   14   15   16   17   18   >