thanks Sunil G for the review..
In ApplicationCLI, public static final String SET_PRIORITY = "setPriority";
Done, changed to updatePriority
In future --appId can be used with other parameters also, correct?
updateApplicationPriority can throw NumberFormatException
Since exception is directly thrown back to client cli, I think this should be fine.
ClientRMService.java has few commented code.
Yes , Since
YARN-3887 was not committed, I was used that patch to compile but while uploading patch I commented for HadoopQA compilation. Now I have uncommented those lines.