[SOLR-3168] "numberToKeep" backups doesn't ever keep more than 1

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 3.5, 4.0-ALPHA
• Fix Version/s: 3.6, 4.0-ALPHA
• Component/s: replication (java)
• Assignee: James Dyer
• Reporter: James Dyer
• Created: 27/Feb/12 21:50
• Updated: 10/May/13 10:40
• Resolved: 28/Feb/12 16:29

Description

On SOLR-2578, Neil Hooey pointed out a silly mistake in SnapShooter.java in that I forgot to increment a counter variable in the case the user wants to keep more than 1 backup.

I will attach a patch for this 1-line fix. I do not intend to amend the unit test as this fix is easy to understand. Also, the test case is already very complex and long-running.

I will commit shortly.