[HBASE-8097] MetaServerShutdownHandler may potentially keep bumping up DeadServer.numProcessing

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.98.0, 0.95.0
• Hadoop Flags: Reviewed
• Assignee: Jeffrey Zhong
• Reporter: Jeffrey Zhong
• Created: 14/Mar/13 00:06
• Updated: 23/Sep/13 18:30
• Resolved: 18/Mar/13 20:02

Description

    } catch (IOException ioe) {
      this.services.getExecutorService().submit(this);
      this.deadServers.add(serverName);
      throw new IOException("failed log splitting for " +
          serverName + ", will retry", ioe);
    }

this.deadServers.add(serverName); will keep incrementing DeadServer.numProcessing

We can't get rid of numProcessing by just checking deadServers.size() because deadServers is also used to report some historically failed RSs.