[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/656 --- 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 is

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129232374 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -275,27 +280,105 @@ The Profiler runs as an independent Storm topology. The configuration

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129232188 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -275,27 +280,105 @@ The Profiler runs as an independent Storm topology. The configuration

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129229395 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -129,23 +129,28 @@ This section will describe the steps required to get your first "Hello,

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129219973 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -17,7 +17,7 @@ Any field contained within a message can be used to generate a profile. A

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129220852 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -83,9 +83,9 @@ At this point the Profiler is running and consuming telemetry messages. We

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129230420 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -275,27 +280,105 @@ The Profiler runs as an independent Storm topology. The configuration

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-25 Thread mattf-horton
Github user mattf-horton commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r129230326 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -129,23 +129,28 @@ This section will describe the steps required to get your first "Hello,

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-19 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r128226967 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -137,15 +137,20 @@ This section will describe the steps required to get your first "Hello,

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-19 Thread MohanDV
Github user MohanDV commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r128193518 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -137,15 +137,20 @@ This section will describe the steps required to get your first "Hello,

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-18 Thread JonZeolla
Github user JonZeolla commented on a diff in the pull request: https://github.com/apache/metron/pull/656#discussion_r128136238 --- Diff: metron-analytics/metron-profiler/README.md --- @@ -277,25 +282,103 @@ The values can be changed on disk and then the Profiler topology must be

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-18 Thread nickwallen
GitHub user nickwallen reopened a pull request: https://github.com/apache/metron/pull/656 METRON-1050 Improve Docs of 'profile.period.duration' It is a common mistake for new user's of the Profiler, to alter the Profiler's `profiler.period.duration` property, but not alter the

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-18 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/656 --- 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 is

[GitHub] metron pull request #656: METRON-1050 Improve Docs of 'profile.period.durati...

2017-07-18 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/656 METRON-1050 Improve Docs of 'profile.period.duration' It is a common mistake for new user's of the Profiler, to alter the Profiler's `profiler.period.duration` property, but not alter the