[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1054 --- 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 enab

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-25 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r54195364 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,65 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of wo

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-25 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r54113951 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,65 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r54042400 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,65 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of wo

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-24 Thread redsanket
Github user redsanket commented on the pull request: https://github.com/apache/storm/pull/1054#issuecomment-188304984 +1 --- 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 a

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-18 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/1054#issuecomment-185813428 @abhishekagarwal87 thanks for the review. I have fixed the type --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-10 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r52448581 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,65 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource u

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-02-05 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/1054#issuecomment-180410973 @redsanket thanks for the review. I have addressed all your concerns --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-29 Thread redsanket
Github user redsanket commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51309621 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-29 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51298506 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-29 Thread redsanket
Github user redsanket commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51293812 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51154779 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/1054#issuecomment-176292755 @HeartSaVioR @unsleepy22 @revans2 thanks for the review! I think I have address all of your comments --- If your project is set up for it, you can reply to this email

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51154587 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51154424 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51139715 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51139298 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51139196 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread jerrypeng
Github user jerrypeng commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51139132 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51136522 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of wo

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-28 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51135789 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of wo

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread unsleepy22
Github user unsleepy22 commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51076681 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread unsleepy22
Github user unsleepy22 commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51076573 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread unsleepy22
Github user unsleepy22 commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51076449 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage of

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/1054#issuecomment-175923449 Took a quick look and looks good. I found some typos which could be easily addressed. --- If your project is set up for it, you can reply to this email and have y

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51072040 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage o

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51071856 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage o

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1054#discussion_r51071834 --- Diff: documentation/cgroups_in_storm.md --- @@ -0,0 +1,61 @@ +# CGroups in Storm + +CGroups are used by Storm to limit the resource usage o

[GitHub] storm pull request: Documentation for cgroup support in Storm

2016-01-27 Thread jerrypeng
GitHub user jerrypeng opened a pull request: https://github.com/apache/storm/pull/1054 Documentation for cgroup support in Storm Related to PR: https://github.com/apache/storm/pull/1053 You can merge this pull request into a Git repository by running: $ git pull https: