[HIVE-13512] Make initializing dag ids in TezWork thread safe for parallel compilation

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.1.0
• Component/s: HiveServer2, Query Planning
• Assignee: Peter Slawski
• Reporter: Peter Slawski
• Created: 14/Apr/16 02:54
• Updated: 21/Jun/16 15:12
• Resolved: 02/May/16 01:20

Description

When parallel query compilation is enabled, it is possible for concurrent running threads to create TezWork objects that have the same dag id. This is because the counter used to obtain the next dag id is not thread safe. The counter should be an AtomicInteger rather than an int.

  private static int counter;
  ...
  public TezWork(String queryId, Configuration conf) {
    this.dagId = queryId + ":" + (++counter);
    ...
  }