[ACCUMULO-3580] TabletStateChangeIterator makes master do unnecessary work

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5.2, 1.6.2
• Fix Version/s: 1.7.0
• Assignee: Christopher Tubbs
• Reporter: Christopher Tubbs
• Created: 11/Feb/15 22:04
• Updated: 26/Feb/15 20:00
• Resolved: 24/Feb/15 21:21

Description

I was examining -Xlint warnings and found this warning about a fall-through case in a switch statement:

The TabletStateChangeIterator in the master does not properly skip over tablets which are hosted and online and have no work to do. This probably makes the master handle these when it doesn't need to.

Issue Links

• breaks: ACCUMULO-3601 master does not process migrations
• breaks: ACCUMULO-3583 TestMergeState.test and ShellServerTest.deleterows broken
• is related to: ACCUMULO-3581 Add more testing for TabletStateChangeIterator
• relates to: ACCUMULO-3618 Tests which invoke `accumulo admin stop ...` fail