[YARN-9593] Updating scheduler conf with comma in config value fails

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.9.0, 3.0.0, 3.2.0, 3.1.2
• Fix Version/s: 3.3.0
• Assignee: Tanu Ajmera
• Reporter: Anthony Hsu
• Created: 01/Jun/19 01:02
• Updated: 26/Feb/20 10:01
• Resolved: 26/Feb/20 09:47

Description

For example:

$ yarn schedulerconf -update "root.gridops:acl_administer_queue=user1,user2 group1,group2"

Specify configuration key value as confKey=confVal.

This fails because there is a comma in the config value and the SchedConfCLI splits on comma first, expecting each split to a k=v pair.

void globalUpdates(String args, SchedConfUpdateInfo updateInfo) {
  if (args == null) {
    return;
  }
  HashMap<String, String> globalUpdates = new HashMap<>();
  for (String globalUpdate : args.split(",")) {
    putKeyValuePair(globalUpdates, globalUpdate);
  }
  updateInfo.setGlobalParams(globalUpdates);
}

Cc: jhung