[TEZ-4186] Limits: Fix init order regression from TEZ-4155

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.10.1
• Fix Version/s: 0.10.0, 0.9.3
• Assignee: Gopal Vijayaraghavan
• Reporter: Gopal Vijayaraghavan
• Created: 20/May/20 23:02
• Updated: 16/Jul/20 15:35
• Resolved: 21/May/20 21:30

Description

After TEZ-4155, the AM is not able to configure Limits parameters correctly, due to the following interaction.

  static {
    init(new TezConfiguration());
  }

prevents setConfiguration from doing anything

  public synchronized static void setConfiguration(Configuration conf) {
    if (Limits.conf == null && conf != null) {
      init(conf);
    }
  }

since the Limits.conf is never null.

Issue Links

• is duplicated by: TEZ-4159 TEZ-4155 prevents setting runtime configuration in Limits