[LUCENE-5080] CMS setters cannot work unless you setMaxMergeCount before you setMaxThreadCount

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.4, 6.0
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Robert Muir
• Created: 27/Jun/13 16:23
• Updated: 28/Aug/22 13:48
• Resolved: 29/Jun/13 01:29

Description

  public void setMaxThreadCount(int count) {
    ...
    if (count > maxMergeCount) {
      throw new IllegalArgumentException("count should be <= maxMergeCount (= " + maxMergeCount + ")");
    }

but:

   public void setMaxMergeCount(int count) {
    ...
    if (count < maxThreadCount) {
      throw new IllegalArgumentException("count should be >= maxThreadCount (= " + maxThreadCount + ")");
    }

So you must call them in a magical order. I think we should nuke these setters and just have a CMS(int,int)