[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/139 --- 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 featu

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-29 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/139#issuecomment-222363512 good catch, I'll make those changes now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-28 Thread james-sirota
Github user james-sirota commented on the pull request: https://github.com/apache/incubator-metron/pull/139#issuecomment-222345898 +1. The only thing I would suggest would be to name the variable parallelism hint and not just parallelism because that name has meaning for someone who

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-28 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/139#issuecomment-222313782 @james-sirota Right you are! Just updated exposing the numTasks for the spout and the parser bolt. ``` -pnt,--parser_num_tasksParser Num Task

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-27 Thread james-sirota
Github user james-sirota commented on the pull request: https://github.com/apache/incubator-metron/pull/139#issuecomment-93764 The settings work, but I think you forgot to expose the parallelism hint. See this blog entry: https://storm.apache.org/releases/1.0.0/Understanding-the-

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-27 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/139#issuecomment-53339 Ok, I verified this on full dev vagrant. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-27 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/139#issuecomment-41649 Just watch out for monit's use of this script. Let me know if you need specific pointers. --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-metron pull request: METRON-190: Make start_parser_topol...

2016-05-27 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/incubator-metron/pull/139 METRON-190: Make start_parser_topology.sh more adaptable regarding storm topology configuration Add the ability to specify the message timeout, the max task parallelism, the number of ac