seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172753133
########## File path: common/src/main/java/edu/snu/nemo/common/ir/Readable.java ########## @@ -28,5 +30,15 @@ * @throws Exception exception while reading data. */ Iterable<O> read() throws Exception; + + /** + * Returns the list of locations where this readable resides. + * Each location has a complete copy of the readable. + * @return List of locations where this readable resides, or an empty list if this operation is not supported + * @throws Exception exceptions on the way + */ + default List<String> getLocations() throws Exception { Review comment: It will introduce some redundant lines in our current codebase, but with long-term view, you have the point. ---------------------------------------------------------------- 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