[ACCUMULO-4361] Test race conditions in ShellServerIT

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.7.3, 1.8.0
• Component/s: test
• Assignee: Josh Elser
• Reporter: Josh Elser
• Created: 08/Jul/16 18:37
• Updated: 11/Jul/16 17:54
• Resolved: 11/Jul/16 03:05

Description

romil.choksi found a test issue where ShellServerIT.getAuths() would intermittently fail.

This is because of the use of ZooCache inside of ZKAuthorizor. The minicluster has two tabletservers. If the shell is so unlucky to update the authorizations on one tserver but then ask for the authorizations on the other tserver, the test is subject to the propagation of the ZK watcher update to fire.

We should just switch over ShellServerIT to run a minicluster with only one tserver than try to hack around this in the test itself with retries.

Issue Links

• relates to: ACCUMULO-2743 Improve stale-reads with ZooKeeper-backed data
• links to: GitHub Pull Request #123
• links to: https://github.com/apache/accumulo/pull/123.patch