johnyangk commented on issue #17: [NEMO-66] Move /tests/scheduler into /runtime/master/scheduler/test URL: https://github.com/apache/incubator-nemo/pull/17#issuecomment-390116743 @jeongyooneo Thanks for the comment. While separating the `testplans` package into two as per your suggestion, I've come to realize that the `BasicPushPolicy` and the `BasicPullPolicy` really should go into the `optimizer` package. So I moved those classes to the `optimizer` package. I've also renamed `testplans` to `plangenerator`, and refactored `TestPlanGenerator` to make its public methods only deal with physical plans, and hide IRs and policies from the tests in the other `runtime` package. At present `TestPlanGenerator` can generate three specific types of plans used in the current scheduler tests, but we can later extend this class to generate many random, complex DAGs to be used by not only the scheduler test but also the test for the executors.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services