Giovanni Matteo Fumarola The patch generally looks good, reasonable refactoring for
Please go over the javadoc and clean them up a bit. Example of small issues:
- Some of the comments only make sense for RM as an implementor, e.g., * @throws NotFoundException if the ResourceScheduler is null
- Some of the text you use misleading capitalizations e.g.,: * @throws Exception in case of a BadRequest where BadRequest is not a java object.
- Comments of the kind Use with @link RMWSConsts#SCHEDULER. are not very helpful. Briefly state what the method does, beside how to use it.
I understand you have in general removed exceptions where were never thrown (generally good). Why adding adding NotFoundException to getSchedulerInfo, getNodes, etc..?
Other than these nits, I am sure yetus would have caught any other issues, so I am good for the patch to get committed. Wangda Tan, since you first reviewedd, do you want to do the commit? (please push to branch-2 as well if possible).