You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by jerrypeng <gi...@git.apache.org> on 2016/10/10 19:25:43 UTC

[GitHub] storm pull request #1733: [STORM-2134] - improving the current scheduling st...

GitHub user jerrypeng opened a pull request:

    https://github.com/apache/storm/pull/1733

    [STORM-2134] - improving the current scheduling strategy for RAS

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jerrypeng/storm STORM-2134

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/1733.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1733
    
----
commit d4e943794458f8177a518b68eb2efe09d48d0c6d
Author: Boyang Jerry Peng <je...@yahoo-inc.com>
Date:   2016-10-06T18:45:55Z

    [STORM-2134] - improving the current scheduling strategy for RAS

commit caf525e63cbb441267b0f7ad4ba51b48675e4fb2
Author: Boyang Jerry Peng <je...@yahoo-inc.com>
Date:   2016-10-06T19:25:51Z

    re-adding accidentally deleted RAS tests

commit 0d616ba539a3e72b7c26fe74a7e8690f7b60a6e8
Author: Boyang Jerry Peng <je...@yahoo-inc.com>
Date:   2016-10-10T03:44:32Z

    adding to documentation

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1733: [STORM-2134] - improving the current scheduling strategy ...

Posted by jerrypeng <gi...@git.apache.org>.
Github user jerrypeng commented on the issue:

    https://github.com/apache/storm/pull/1733
  
    @revans2 thanks for the reviews.  I have addressed your comments


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request #1733: [STORM-2134] - improving the current scheduling st...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/storm/pull/1733


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1733: [STORM-2134] - improving the current scheduling strategy ...

Posted by jerrypeng <gi...@git.apache.org>.
Github user jerrypeng commented on the issue:

    https://github.com/apache/storm/pull/1733
  
    Also re-add some RAS related tests that were accidentally deleted in the past


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request #1733: [STORM-2134] - improving the current scheduling st...

Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1733#discussion_r82810383
  
    --- Diff: storm-core/test/jvm/org/apache/storm/scheduler/resource/strategies/eviction/TestDefaultEvictionStrategy.java ---
    @@ -0,0 +1,736 @@
    +package org.apache.storm.scheduler.resource.strategies.eviction;
    --- End diff --
    
    Needs a license header.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1733: [STORM-2134] - improving the current scheduling strategy ...

Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/1733
  
    +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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---