jeongyooneo commented on a change in pull request #17: [NEMO-66] Move /tests/scheduler into /runtime/master/scheduler/test URL: https://github.com/apache/incubator-nemo/pull/17#discussion_r187842301
########## File path: pom.xml ########## @@ -63,6 +63,7 @@ limitations under the License. <module>runtime/executor</module> <module>runtime/master</module> <module>runtime/driver</module> + <module>runtime/testplans</module> Review comment: Since it contains both scheduling policies and generating PhysicalPlans, it would be good to rename `testplans` to `runtimetest` and split it into separate packages, for example `testpolicy` and `testplan` for reusability. ---------------------------------------------------------------- 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