[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/977 ---

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-06 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r179871500 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/DefaultMessageDistributor.java --- @@ -262,11 +262,19 @@ public

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-06 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r179868541 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/DefaultMessageDistributor.java --- @@ -262,11 +262,19 @@

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-06 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r179860237 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/DefaultMessageDistributor.java --- @@ -262,11 +262,19 @@ public

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178600804 --- Diff: metron-analytics/metron-profiler/src/test/java/org/apache/metron/profiler/integration/ProfilerIntegrationTest.java --- @@ -70,247 +66,103 @@

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178600343 --- Diff: metron-analytics/metron-profiler/src/main/java/org/apache/metron/profiler/bolt/ProfileBuilderBolt.java --- @@ -310,17 +313,37 @@ public void

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178600386 --- Diff: metron-analytics/metron-profiler/src/main/java/org/apache/metron/profiler/bolt/ProfileBuilderBolt.java --- @@ -339,11 +362,13 @@ private void

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178600285 --- Diff: metron-analytics/metron-profiler/src/main/java/org/apache/metron/profiler/bolt/ProfileBuilderBolt.java --- @@ -310,17 +313,37 @@ public void

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178599870 --- Diff: metron-analytics/metron-profiler/src/main/java/org/apache/metron/profiler/bolt/ProfileBuilderBolt.java --- @@ -395,10 +420,46 @@ private void

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178599674 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/DefaultMessageDistributor.java --- @@ -281,29 +289,45 @@

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/977#discussion_r178599132 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/DefaultMessageDistributor.java --- @@ -262,11 +262,19 @@

[GitHub] metron pull request #977: METRON-1505 Intermittent Profiler Integration Test...

2018-04-02 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/977 METRON-1505 Intermittent Profiler Integration Test Failure ### Problem The integration tests were failing intermittently when Storm unexpectedly expired messages generated by the