You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2020/10/20 20:55:02 UTC
[GitHub] [storm] bipinprasad opened a new pull request #3340: [STORM-3706] Add Cluster.needsSchedulingRas() to tests
bipinprasad opened a new pull request #3340:
URL: https://github.com/apache/storm/pull/3340
## What is the purpose of the change
*Prior to refactoring of Resource Aware Scheduler, Cluster.needsSchedulingRas() would always return true. Call this method in tests to ensure that the correct value (true/false as appropriate) is returned.*
## How was the change tested
*Run unit tests, especially TestResourceAwareScheduler and TestBackwardCompatibility*
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [storm] bipinprasad commented on pull request #3340: [STORM-3706] Add Cluster.needsSchedulingRas() to tests
Posted by GitBox <gi...@apache.org>.
bipinprasad commented on pull request #3340:
URL: https://github.com/apache/storm/pull/3340#issuecomment-713201311
License problem is back. Closing and reopening for recompile.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [storm] Ethanlm merged pull request #3340: [STORM-3706] Add Cluster.needsSchedulingRas() to tests
Posted by GitBox <gi...@apache.org>.
Ethanlm merged pull request #3340:
URL: https://github.com/apache/storm/pull/3340
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [storm] bipinprasad closed pull request #3340: [STORM-3706] Add Cluster.needsSchedulingRas() to tests
Posted by GitBox <gi...@apache.org>.
bipinprasad closed pull request #3340:
URL: https://github.com/apache/storm/pull/3340
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org