[HIVE-20015] Populate ArrayList with Constructor

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 3.0.0, 4.0.0
• Fix Version/s: 4.0.0-alpha-1
• Component/s: HiveServer2
• Labels: newbie noob
• Assignee: Daniel Voros
• Reporter: David Mollitor
• Created: 27/Jun/18 15:33
• Updated: 17/Nov/22 08:54
• Resolved: 25/Jul/18 22:27

Description

MapWork.java

  public ArrayList<Operator<?>> getWorks() {
    return new ArrayList<Operator<?>>(aliasToWork.values());
  }

  public ArrayList<Path> getPaths() {
    ArrayList<Path> ret=new ArrayList<>();
    ret.addAll(pathToAliases.keySet());
    return ret;
  }

getWorks method correctly uses the constructor to populate the ArrayList. Please update getPaths method to do the same, instead of creating an empty array, then creating a new array to accommodate the addAll request.